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

Documentatie: toegankelijkheid role="group" voor form field #522

Open
2 tasks
Robbert opened this issue Oct 30, 2023 · 3 comments
Open
2 tasks

Documentatie: toegankelijkheid role="group" voor form field #522

Robbert opened this issue Oct 30, 2023 · 3 comments

Comments

@Robbert
Copy link
Member

Robbert commented Oct 30, 2023

De form field van Utrecht is op dit moment een <div> zonder accessible role en zonder label. De component van RVO voegt daar code aan toe: `role="group" aria-labelledby="...".

Is dit een toegankelijk patroon? Ik ken role="group" alleen voor fieldsets. Als dit niet gewenste aanpak is, dan moet dit in de documentatie van form field worden toegevoegd.

  • controleren aanpak role="group" voor form fields
  • documentatie schrijven over of dit ongewenste markup is, of dat het alternatieve correcte markup is. Ik zou de documentatie toevoegen bij de form field component van Utrecht.
@rianrietveld
Copy link

rianrietveld commented Oct 30, 2023

Een role="group" hier plaatsen (om een label/input) met een aria-describedby is hier overbodig. De label/form velden zijn al met elkaar vebonden van for/id. De div is hier puur voor de layout. Een role plus aria-label/aria-describedby zorgt ervoor dat de naam twee keer wordt uitgesproken en dat lijkt me zinloze extra informatie.
First rule of ARIA ....

Niet doen dus, voor een label/input, hier de for/id relatie is voldoende voor het begrip van het formulierveld.
Wil je meerdere formuliervelden grouperen, doe dat dan met een fieldset/legend en niet met een role="group".

Voor het groeperen van button aan het einde van een formulier kan een <ul><li> lijst gebruikt worden.

@sushitommy
Copy link
Contributor

Thanks voor de input, ik ga dit verwerken.

@Yolijn Yolijn added this to the Missing form elements milestone Jan 21, 2024
@Yolijn
Copy link
Member

Yolijn commented Jan 21, 2024

Voorstel: verplaats dit issue naar discussions om input uit de community op te halen

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

No branches or pull requests

4 participants