DatePicker - multiple validators should render separate helper texts
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 35/100
- Tipo de issue
- Funcionalidade
- Clareza
- Razoavelmente clara
- Status de atividade
- Estagnada
- Stack de tecnologia
- react, typescript
- Domínio
- frontend
Direção de pesquisa
Comece localizando o ponto de entrada de DatePicker e sua renderização atual de HelperText, além do tratamento de validator. Confirme como várias strings de erro de validator são representadas e, em seguida, faça com que o comportamento concluído tenha um item HelperText separado para cada string de erro retornada e verifique o comportamento de DatePicker.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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?
- Linguagem predominante
- TypeScript
- Estrelas
- 862
- Forks
- 392
- Merge médio
- 4d 8h
- PRs com merge (30d)
- 9
Guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de patternfly/patternfly-react
-
PF Team
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
patternfly/patternfly-react#12651 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
patternfly/patternfly-react#12637 ·
-
PF Team
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
patternfly/patternfly-react#12617 · 1 comentário ·
-
PF Team
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
patternfly/patternfly-react#12610 · 1 comentário · 1 reação ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
patternfly/patternfly-react#12600 ·
Todas as issues de patternfly/patternfly-react
Issues semelhantes
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 84/100
bcgov/bc-wallet-mobile#4761 · 1 comentário ·
-
external-issue to-triage
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 88/100
-
area-deployment area-integrations triage:bot-seen
Dificuldade 2/5 Meio dia Facilidade para iniciantes 86/100
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
-
refactor
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 84/100