DatePicker - multiple validators should render separate helper texts
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 35/100
- Type d'issue
- Fonctionnalité
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- react, typescript
- Domaine
- frontend
Piste de recherche
Commencez par localiser le point d’entrée de DatePicker ainsi que son rendu actuel de HelperText et la gestion de validator. Vérifiez comment sont représentées plusieurs chaînes d’erreur de validator, puis faites en sorte que le comportement final comporte un élément HelperText distinct pour chaque chaîne d’erreur renvoyée et vérifiez le comportement de DatePicker.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Is this a new component or an extension of an existing one?
DatePicker
Describe the feature
Currently DatePicker renders only a single HelperText for errors, regardless of how many validators may be passed in. If multiple validators return a error string, each string is added to a single string state with an escaped newline between them, resulting in a single helper text item (with the intent being each validator string will be separated by a new line, but it doesn't seem like that is happening for me, i.e. it all gets rendered as a single line).
Instead we should create a new HelperText item for each validator that returns an error string so as to be less confusing if multiple errors are being rendered.
CC @andrew-ronaldson is this the intended behavior, or is there something else in mind for multiple validators returning error text?
Are there visuals for this feature? If applicable, please include examples for each state and for varying widths
Include screenshots or links to Marvel or other mockups.
Any other information?
- Langage dominant
- TypeScript
- Étoiles
- 862
- Forks
- 392
- Merge moyen
- 4 j 8 h
- PR mergées (30 j)
- 9
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de patternfly/patternfly-react
-
PF Team
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
patternfly/patternfly-react#12651 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
patternfly/patternfly-react#12637 ·
-
PF Team
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
patternfly/patternfly-react#12617 · 1 commentaire ·
-
PF Team
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
patternfly/patternfly-react#12610 · 1 commentaire · 1 réaction ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
patternfly/patternfly-react#12600 ·
Toutes les issues de patternfly/patternfly-react
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
bcgov/bc-wallet-mobile#4761 · 1 commentaire ·
-
external-issue to-triage
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
-
area-deployment area-integrations triage:bot-seen
Difficulté 2/5 Une demi-journée Accessibilité débutants 86/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
-
refactor
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100