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

Language selector ARIA implementation #436

Open
ppvg opened this issue Dec 6, 2023 · 1 comment
Open

Language selector ARIA implementation #436

ppvg opened this issue Dec 6, 2023 · 1 comment
Labels
a11y Accessibility fixes and improvements Core set From the base set of components Header Component 'Header' Navigation Component 'Navigation'
Milestone

Comments

@ppvg
Copy link
Member

ppvg commented Dec 6, 2023

From independent accessibility report, SC 3.3.2:

Op pagina components/header-example-language-select-list.html staat een element waarmee een taal geselecteerd kan worden. Deze heeft visueel geen label. Omdat het op zichzelf voldoende duidelijk is wat er gebeurt als er een taal wordt geselecteerd, is dit ook niet verplicht. Het kan echter wel om de functie nog iets duidelijker te maken. In de code is visueel verborgen tekst aanwezig, maar deze is niet gekoppeld aan het veld, doordat het aria-describedby attribuut op een div-element staat.

From independent accessibility report, SC 4.1.2:

Op pagina components/header-example-language-select-list.html staat een uitklapbaar element waarmee een andere taal geselecteerd kan worden. Dit is nu een knop (button-element). Het aria-expanded-attribuut, waarmee wordt aangegeven wat de waarde (in- of uitgeklapt) is, staat echter op een div-element dat eromheen staat. Dit attribuut moet op het interactieve element staan, het element dat daadwerkelijk wordt in- en uitgeklapt. Deze informatie wordt hierdoor nu niet goed overgebracht.

The language selector places aria-describedby and aria-expanded on the wrapper div. It should instead be on the interactive element, the button.

Additionally, we should look into whether the role="listbox" is correct. It might make sense to implement the ARIA APG's Navigation Menu Button Example here instead.

@ppvg
Copy link
Member Author

ppvg commented Dec 20, 2023

Additional comments: when using role="listbox", the child elements should also get the appropriate role, "option". See Listbox Pattern (apg).

@ppvg ppvg added the a11y Accessibility fixes and improvements label May 9, 2024
@reinschaap reinschaap added Navigation Component 'Navigation' Header Component 'Header' labels Sep 26, 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 Core set From the base set of components Header Component 'Header' Navigation Component 'Navigation'
Projects
None yet
Development

No branches or pull requests

3 participants