Button - update console error for accessible name and move aria-expanded location
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 55/100
- Tipo de issue
- Funcionalidade
- Clareza
- Razoavelmente clara
- Status de atividade
- Estagnada
- Stack de tecnologia
- react, typescript
- Domínio
- accessibility, frontend
Direção de pesquisa
Comece revisando as alterações referenciadas no pull request #11861, especialmente o erro de console de accessible-name por volta da linha 164 e o tratamento de aria-expanded por volta da linha 239. Localize os testes relacionados de Button mencionados na issue, atualize-os para o comportamento solicitado e confirme que o tratamento do erro e do atributo corresponde aos requisitos declarados.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
Describe the enhancement or change
Followup to some comments left in https://github.com/patternfly/patternfly-react/pull/11861/
- The console error with the
isSettings || isHamburger || isFavoriteconditional for the accessible name (line 164 in that PR) should be updated to remove that conditional. We should be throwing an error for any button that doesn't have children, aria-label, or aria-labelledby - The
aria-expandedattribute (line 239) should be moved below the{...props}so that it won't be overridden by a manualaria-expandedbeing passed in - Several tests related to the above points will need to be updated in some fashion as well
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?
- 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