Skip to content

Conversation

@1Copenut
Copy link
Contributor

Changed the aria-busy fail message per guidance in issue #3934.

Closes:
This PR closes #3934.

@1Copenut 1Copenut requested a review from a team as a code owner March 23, 2023 18:30
@WilcoFiers WilcoFiers changed the title Updating aria-busy error message. chore: Updating aria-busy error message. Mar 29, 2023
Copy link
Contributor

@WilcoFiers WilcoFiers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All good. Thanks for the PR @1Copenut!

@WilcoFiers
Copy link
Contributor

Reviewed for security.

@WilcoFiers WilcoFiers merged commit 3b190a2 into dequelabs:develop Mar 29, 2023
@1Copenut 1Copenut deleted the fix/aria-busy-remediation-message branch March 29, 2023 19:25
rafaelpaniago1 added a commit to masganem/axe-core-mes-20252 that referenced this pull request Nov 7, 2025
Implementação da correção da mensagem de remediação confusa
do check aria-busy.

Problema:
- Mensagem original: 'Element has no aria-busy="true" attribute'
- Não explicava QUANDO usar o atributo aria-busy

Solução aplicada:
- Nova mensagem: 'Element uses aria-busy="true" while showing a loader'
- Fornece contexto claro de quando aplicar a correção
- Arquivo modificado: lib/checks/aria/aria-busy.json

Comparação com solução oficial (PR dequelabs#3954):
✅ IDÊNTICA - As soluções são 100% iguais!
- Mesmo texto
- Mesmo arquivo
- Mesma abordagem

Esta foi uma correção simples mas importante para melhorar
a experiência do desenvolvedor com mensagens mais claras
e acionáveis.

Análise completa em: ANALISE-ISSUE-3934.md

Referências:
- Issue: dequelabs#3934
- PR oficial: dequelabs#3954
- Commit oficial: 3b190a2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Clarify aria-busy remediation message

2 participants