-
Notifications
You must be signed in to change notification settings - Fork 9
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
[a11y] Announce table row count to ATs #12429
Conversation
I tried to work through things with Microsoft Narrator and it just made some things like buttons and inputs non-functional. Well, I shouldn't be too surprised |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple of things noticed
- When using the search bar on a client-side table, nothing is read out for a results change. This works on the other hand for server-side tables
- Navigating to a server-side table fires the results count twice, first zero results then the actual count. Probably the before and after of the query
- Cannot interact with "Filter by" button when NVDA is active
Thanks! I was able to fix these two (sort of) in fix announcement for client side tables. I got it working on client side, however it does announce it once page loads which I feel as though maybe we can live with since it is not absurd to say like "Heading level one, users, X items found". Weird, but probably worth more effort than it is worth to try and fix this. 🤷♀️ If anyone disagrees I can look into fixing that as well.
Wow, I can't beleive no one has noticed this 😬 Totally a massive issue and a ticket has been created, #12454. Feel free to add any details I may have missed 👍 |
It no longer announces the total when navigating to a page for me, but it does announce it when updated by filters or search bar.
🙃 |
defaultMessage: | ||
"{total, plural, =0 {0 results found} =1 {1 result found} other {# result found}}", | ||
id: "uZYkk1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This isn't working right, it is reading out "total, plural..." for me
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops, forgot the variable name was not total 😅
Fixed in fix intl variable name
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't
other {# result found}}
Be
other {# results found}}
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
okay, I tihnk I finally got it 😆
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay... As far as I can tell I think things are good!
🤖 Resolves #12100
👋 Introduction
Adds an announcement for ATs that the total number of items in the table has changed.
🧪 Testing
pnpm run dev:fresh
/skills
[email protected]
/admin/users