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

Onnodige "arialabeledby" -label + aria-live #689

Open
savitris opened this issue Apr 30, 2024 · 6 comments · Fixed by #708
Open

Onnodige "arialabeledby" -label + aria-live #689

savitris opened this issue Apr 30, 2024 · 6 comments · Fixed by #708
Assignees
Labels
a11y wcag/1.3.1 Info and Relationships wcag/4.1.2 Name, Role, Value

Comments

@savitris
Copy link
Contributor

savitris commented Apr 30, 2024

Pagina: PDC Online Loket Homepagina

Probleem: De Searchbar heeft een aria-label en een arialabeledby label (downshift-18-label).

Potentiële oplossing:
Alleen de aria-label is genoeg om te hebben.

Extra: Voeg toe aria-live="polite" zodat de Screenreader gebruiker updates in een beter tempo kan ontvangen (van de item suggesties).

Gerelateerde afbeelding:
Screenshot 2024-04-30 at 11 16 10

Succes Criteria:
1.3.1
4.1.2

@savitris savitris added this to the A11y PDC Digitaal Loket milestone Apr 30, 2024
@Robbert Robbert added the a11y label May 1, 2024
@AliKdhim87 AliKdhim87 self-assigned this May 3, 2024
@AliKdhim87
Copy link
Contributor

Ik heb de zoekbalk aangepast zodat alleen het aria-labelledby attribuut behouden blijft. Dit zorgt voor een duidelijke koppeling met het label, wat de toegankelijkheid verbetert voor gebruikers van assistive technologies.

Op basis van de implementatie van de zoekbalk lijkt het niet noodzakelijk om het aria-live="polite" attribuut toe te voegen. Het aria-live attribuut wordt doorgaans gebruikt om assistive technologies te informeren dat de inhoud binnen een element dynamisch kan veranderen en aan de gebruiker moet worden aangekondigd.

In ons geval lijkt de autocomplete-functionaliteit al goed gestructureerd met passende rollen en attributen, zoals role="listbox", role="option", en aria-selected, die voldoende informatie bieden aan assistive technologies over de dynamische wijzigingen in de autocomplete-lijst.

@AliKdhim87 AliKdhim87 linked a pull request May 3, 2024 that will close this issue
@savitris savitris reopened this May 16, 2024
@savitris
Copy link
Contributor Author

@AliKdhim87
De aria-labelledby="downshift-382-label" wordt niet gelezen. Waar komt de waarde downshift-382-label vandaan?
Op dit moment, ook al is er een aria-labelledby, wordt er nog steeds van uitgegaan dat de component geen toegankelijke tekst (Accessible Name) bevat.

@Robbert
Copy link
Member

Robbert commented May 16, 2024

Gebruik liever aria-labelledby dan aria-label, voor betere Google Translate support.

@AliKdhim87
Copy link
Contributor

@savitris Met betrekking tot het lezen van de aria-labelledby="downshift-382-label" attribuut, welke browser heb jij gebruikt? De waarde van de aria-labelledby wordt automatisch gegenereerd door het downshift-pakket. Als je de Safari-browser gebruikt, lijkt het erop dat deze een bug heeft: downshift-js/downshift#1539

@AliKdhim87
Copy link
Contributor

@savitris
Copy link
Contributor Author

savitris commented May 16, 2024

@savitris Met betrekking tot het lezen van de aria-labelledby="downshift-382-label" attribuut, welke browser heb jij gebruikt? De waarde van de aria-labelledby wordt automatisch gegenereerd door het downshift-pakket. Als je de Safari-browser gebruikt, lijkt het erop dat deze een bug heeft: downshift-js/downshift#1539

@AliKdhim87 Helaas zie ik het probleem ook op Chrome, Edge en Firefox.

@Robbert Robbert added wcag/1.3.1 Info and Relationships wcag/4.1.2 Name, Role, Value labels May 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y wcag/1.3.1 Info and Relationships wcag/4.1.2 Name, Role, Value
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants