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

VOTE-1019: Add Field grouping for screen readers #273

Open
wants to merge 5 commits into
base: stage
Choose a base branch
from

Conversation

rkoch2
Copy link
Contributor

@rkoch2 rkoch2 commented Aug 5, 2024

This ticket aims to accomplish two things:

  1. Add an aria-label to the blue informative alert so that a screen reader will read its text contents (before, the screen reader would skip over this element completely)
  2. Add field groupings (eg. to all of the name-related fields on the Personal information page) for screen readers

To test, open the NVRF form filler and run WAVE. The blue informative alert should have an aria-label, like below:
image

The associated fields should be contained within a

with role "group" and an appropriate aria-label, like below:
image

Repeat this for the Personal Information and Address pages (which have both informative alerts and Fieldsets). Also check the informative alerts on the Identification and Political party pages.

@rkoch2 rkoch2 requested a review from clmedders August 9, 2024 14:31
@rkoch2 rkoch2 marked this pull request as ready for review August 15, 2024 21:09
@rkoch2 rkoch2 requested a review from clmedders August 15, 2024 21:09
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

Successfully merging this pull request may close these issues.

2 participants