-
-
Notifications
You must be signed in to change notification settings - Fork 654
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
Aria-selected is not narrated for the gridcell #11986
Comments
Hi Team, is there any update on this bug? |
cc: @SaschaCowley |
This is because of bugs in Firefox and Chromium.
|
I wasn't aware of this; thanks.
|
I filed https://bugzilla.mozilla.org/show_bug.cgi?id=1894436 for gridcell. I also filed https://bugzilla.mozilla.org/show_bug.cgi?id=1894437 for option, tab and treeitem. However, I see that Chromium has the same bug and I honestly wonder whether the spec needs to be changed here because supporting that rule is almost certainly going to cause hideous performance problems in some cases. |
I see the selected state isn't being reported for the selected date in Firefox when it's focused, but I'm struggling to understand why. Even if Firefox is incorrectly exposing the selected state when it shouldn't in some cases, why should that prevent the selected state from being reported? It's... very confusing. |
I think this is because NVDA doesn't report "selected" for the selected item in single-select controls. Though honestly I find NVDA's reporting of "selected" and "not selected" to be irregular and confusing. |
The fact that it incorrectly reports "selected" when you move to a (not selected) date would suggest otherwise. Also, this isn't based on single or multi select controls; it's based on the role. If I'm reading the code correctly, selected should get reported for all selectable table cells unconditionally. |
Ah, this is deliberate and based on the selected item count. If only 1 item is selected, the selected state isn't reported. |
...
Yup, so because Firefox is reporting the focused item as selected, if there is a cell with |
We could try working around this in NVDA by checking if any selection widgets contain an explicit selection, and if so removing the selected state from any children without |
We'll fix the Firefox bug. But my point is that this won't solve the reporter's original request:
It explicitly won't say selected because the selection count will now be 1 instead of 2. So we'd be fixing a bug, yes, but it would be a different issue to the one that's reported here. The issue reported here, as I understand it, is intended behaviour (not reporting "selected"). |
I interpreted 'it narrates "selected" state as well' to mean that the selection state is accurately reported, not necessarily that "selected" is spoken. We closed a (somewhat) similar issue (#16449) yesterday as expected behaviour. If your interpretation is correct, I personally think that a new issue should be created, as the problem applies to more than just ARIA gridcells. |
I'll leave that call up to you. For what it's worth, I've submitted a patch to fix the Firefox gridcell issue. |
The Firefox gridcell bug has been fixed in Firefox Nightly and the fix will be included in the Firefox 127 release. |
Anyone think maybe the https://dequelabs.github.io/combobo/demo/ When |
In Firefox this now seems to be reported properly. Not selected dates are reported as "not selected" and the selected date is reported without any selection atribute which is clear enough that a selection has been performed. @alexstine could you please open a new issue with your behavior, explaining exactly what happens? When do you exactly set aria selected? I suspect this happens after you press enter or when you click on an item. Before that no aria selected is set at all so NVDA cannot report anything. |
@Adriani90 You are correct. Before a selection is made with the enter key, |
Result on the chromium bug, it seems it is an NVDA issue. NVDA reports the selected value only when using ctrl+alt+arrow keys in browse mode. cc: @SaschaCowley the Chromium issue has been closed since it is NVDA specific. Note that in Firefox, NVDA announces the currently selected item withut atribute, and the unselected items as "unselected" when using arrow keys in focus mode. This is expected behavior in Chrome as well. Is Chrome not exposing the right things here? |
As @jcsteh has pointed out, this is due to incorrect authoring. All selectable gridcells have to have |
Steps to reproduce:
Actual behavior:
For example, when 20th of January was selected narration was:
"January 20, 2021 row 5"
Expected behavior:
it narrates "selected" state as well
Expecting this narration, because:
and
https://www.w3.org/TR/wai-aria-1.1/#aria-selected
Snippet of HTML of element which is focused:
<button role="gridcell" aria-selected="true" data-aa-class="DatepickerCalendarCellButton" class="ui-datepicker__calendarcellbutton il iz ja bg bh jb jc jd je iu ip iv ir jf jg jh ji is as bp bq br bs bt bu bv bw bx by bz ca gs gt gu gv cf cg ch ci cj ck cl cm cn co cp cq cr cs ct cu cv gw gx gy gz da db dc dd de df dg dh di dj jn jm iw ix" aria-label="January 20, 2021" tabindex="0">20</button>
System configuration
NVDA installed
NVDA version: 2020.3
Windows version:
10.0.19042
Name and version of other software in use when reproducing the issue:
Chrome Version 87.0.4280.88 (Official Build) (64-bit)
Other information about your system:
Other questions
Does the issue still occur after restarting your computer?
yes
Have you tried any other versions of NVDA? If so, please report their behaviors.
no haven't
If addons are disabled, is your problem still occuring?
no addons enabled
Did you try to run the COM registry fixing tool in NVDA menu / tools?
no
The text was updated successfully, but these errors were encountered: