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

Table sorting - screen reader not announcing on sort #3538

Closed
1 task
ataker opened this issue Nov 20, 2024 · 0 comments · Fixed by department-of-veterans-affairs/component-library#1433
Closed
1 task
Assignees
Labels
accessibility Any Section 508 or accessibility issue component-update Change to an existing component platform-design-system-team va-table DS Table component

Comments

@ataker
Copy link
Contributor

ataker commented Nov 20, 2024

Description

Continuation of Table sorting - accessibility fixes #3455.
Per the previous ticket:

Screen readers (all) - does not announce when the table is sorted. Currently, it sorts, but then reads out the button's new title attribute. We have aria-live="polite" SR text in there, but I believe focusing the button is overriding the reading of that text. I believe we do that because of a re-render of the table. We do want to maintain focus on the button, but I'm not sure how we can avoid re-rendering.

NVDA + Firefox & Narrator + Edge - sorting the table will repeat the table description "this is a sortable table with 5 rows and 8 columns", then reads the button title. I believe this is also from re-rendering.

In my testing, I wasn't able to get the aria-live="polite" text to announce even after removing the button focusing. My guess is something about the re-render is causing the aria-live="polite" to be considered a new element instead of an updated element, so it is not announced.

Tasks

  • Screen reader announces when the table is sorted
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility Any Section 508 or accessibility issue component-update Change to an existing component platform-design-system-team va-table DS Table component
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant