patternfly / patternfly/patternfly-react

TextInput/TextinputGroup - consistent status/validation prop naming

Ouverte
#10,851 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Breaking change :boom: PF Team
Langage dominant
TypeScript
Étoiles
862
Forks
392
Merge moyen
4 j 8 h
PR mergées (30 j)
9

Description

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

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par la discussion de la pull request liée et les définitions des props de TextInput et TextinputGroup. Examinez les autres composants dotés de props d’état ou de validation, puis tranchez les questions de nommage et de valeurs autorisées ; la tâche est terminée lorsque les conventions sont convenues et appliquées de manière cohérente.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
react, typescript
Domaine
design, frontend
Type d'issue
Fonctionnalité
Difficulté
5/5
Temps estimé
Plus d'une semaine
Activité
À l'abandon
Clarté
À clarifier
Accessibilité débutants
25/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.