Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Collapsible wrapper incorrect aria-label #434

Open
ppvg opened this issue Dec 6, 2023 · 0 comments
Open

Collapsible wrapper incorrect aria-label #434

ppvg opened this issue Dec 6, 2023 · 0 comments
Labels
a11y Accessibility fixes and improvements Collapsible Component 'Collapsible menu' Core set From the base set of components
Milestone

Comments

@ppvg
Copy link
Member

ppvg commented Dec 6, 2023

From independent accessibility report, SC 2.5.3:

Op pagina components/collapsible.html staan knoppen waarmee een menu uitgeklapt kan worden. Deze staan in een div met daarop een aria-label met de tekst "Acties". Een aria-label op een div doet niets en dit aria-label heeft nu dus geen functie. Als deze echter op een interactief element, bijvoorbeeld op de knop, zouden staan, dan zou deze tekst de visueel zichtbare tekst overschrijven in de toegankelijkheidsnaam. Dat zou dan een probleem zijn voor dit succescriterium. Let hier dus op bij het gebruik van aria- label.

The Collapsible's wrapper should not have an aria-label.

@ppvg ppvg added the a11y Accessibility fixes and improvements label Dec 6, 2023
@HeleenSG HeleenSG added the Collapsible Component 'Collapsible menu' label Sep 19, 2024
@Davidemeer Davidemeer added the Core set From the base set of components label Sep 30, 2024
@Davidemeer Davidemeer added this to the Fase 2 Manon milestone Oct 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y Accessibility fixes and improvements Collapsible Component 'Collapsible menu' Core set From the base set of components
Projects
None yet
Development

No branches or pull requests

3 participants