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

[calendar] Date pronounced twice in NVDA #193

Closed
erikkroes opened this issue Jul 22, 2019 · 2 comments
Closed

[calendar] Date pronounced twice in NVDA #193

erikkroes opened this issue Jul 22, 2019 · 2 comments
Labels
a11y Related to A11Y bug Something isn't working

Comments

@erikkroes
Copy link
Collaborator

erikkroes commented Jul 22, 2019

Using NVDA and Firefox, when you navigate the calendar grid, dates get pronounced twice. For example:

13 July 2019 Saturday not selected row 3 Saturday column 7
13 July 2019 Saturday button

This is rather verbose and not preferred.

The first part seems to be the accessible name of the cell, even though the "not selected" comes from an aria attribute on the button.

It gets less verbose when role="gridcell" is applied but it's not a solution. Also because the role is there for a reason. Removing role="application" might also help, but removal would creates issues with keyboard navigation. Further investigation is needed.

This pronounciation might be a bug or design choice from NVDA. I think it would be an improvement for the component if it would be able to circumvent this and be less verbose.

@erikkroes erikkroes added the a11y Related to A11Y label Jul 22, 2019
@erikkroes erikkroes added the enhancement New feature or request label Nov 13, 2019
@jorenbroekema jorenbroekema added the votes needed Enhancement available for voting label Dec 5, 2019
@daKmoR daKmoR reopened this Sep 30, 2020
@daKmoR daKmoR added bug Something isn't working and removed enhancement New feature or request votes needed Enhancement available for voting labels Sep 30, 2020
@tlouisse
Copy link
Member

tlouisse commented Apr 26, 2021

Did quite some testing here (also tried to render everything to light dom, like in the w3c grid example) and had the same findings as @erikkroes

Maybe we can get rid of the button and apply tabindex="0" and all aria attributes on the element with [role=gridcell]?
It supports aria-pressed and aria-current:
https://www.w3.org/TR/wai-aria-1.1/#gridcell

@daKmoR
Copy link
Collaborator

daKmoR commented May 12, 2021

after many more tests I'm pretty confident that it's actually a bug in NVDA 🙈

Raise an issue there
nvaccess/nvda#12392

@Knorgias Knorgias closed this as not planned Won't fix, can't repro, duplicate, stale Jul 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y Related to A11Y bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants