patternfly / patternfly/patternfly-react

TextInput/TextinputGroup - consistent status/validation prop naming

Abierto
#10,851 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Breaking change :boom: PF Team
Lenguaje dominante
TypeScript
Estrellas
862
Forks
392
Merge medio
4 d 8 h
PR fusionados (30 d)
9

Descripción

Describe the enhancement or change
Related to convo chain at https://github.com/patternfly/patternfly-react/pull/10815#discussion_r1697713753

Would also pertain to other components that have a status/validation type prop. We should align on:

  • whether the prop should be status or validation (personally would lean towards status)

  • whether the prop type should only include valid class values, e.g. success, error/danger (another thing to align on), warning, info, and custom, or whether there should also be a "default" value to sort of unset it. If the latter, then does "default" make sense or should it be "none" or something else?

Is this request originating from a Red Hat product team? If so, which ones and is there any sort of deadline for this enhancement?

Any other information?


Jira Issue: PF-32

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Comienza con la discusión del pull request enlazado y las definiciones de props de TextInput y TextinputGroup. Revisa otros componentes con props de estado o validación y, después, resuelve las cuestiones sobre los nombres y los valores permitidos; se considera terminado cuando las convenciones estén acordadas y se apliquen de forma coherente.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
react, typescript
Área
design, frontend
Tipo de issue
Nueva funcionalidad
Dificultad
5/5
Tiempo estimado
Más de una semana
Estado de actividad
Estancado
Claridad
Necesita aclaración
Aptitud para principiantes
25/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.