processing / processing/p5.js-web-editor
Screen reader says the wrong count in Preferences modal due to missing `radiogroup`
@lindapaiste is already working on this.
Since Nov 28, 2023.
- Dominant language
- JavaScript
- Stars
- 1.7k
- Forks
- 1.7k
- Avg merge
- 3d 4h
- Merged PRs (30d)
- 8
Description
Increasing Access
We want the editor to be fully navigable using screen readers.
Feature enhancement details
I downloaded a screenreader and I'm playing around and finding issues. In the preferences modal it announces like this:
"Autosave on checked radio button 4 of 11"
It should be announcing "1 of 2" instead of "4 of 11" (where 1 is "on" and 2 is "off"). But it uses the total count of the radio buttons in the form because we do not have the radiogroup attribute around each "on/off" section.
Might make sense to address this after #2323 so that we only need to change it in one (or two) places instead of many.
Resources:
https://www.w3.org/WAI/ARIA/apg/patterns/radio/examples/radio/
https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Roles/radiogroup_role
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.