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

aria-live updates related to the accessible view overlap with the announcement of the focused control when navigating the VS Code UI with NVDA #199959

Closed
rperez030 opened this issue Dec 4, 2023 · 5 comments · Fixed by #199978 or #199988
Assignees
Labels
accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues bug Issue identified by VS Code Team member as probable bug candidate Issue identified as probable candidate for fixing in the next release verified Verification succeeded windows VS Code on Windows issues
Milestone

Comments

@rperez030
Copy link
Contributor

rperez030 commented Dec 4, 2023

Type: Bug

When navigating the UI using the keyboard, NVDA reads the announcement related to the accessible view before the controls that receives focus. I'm finding this to be quite a bad experience, and potentially a big issue for more beginner users of the editor. An easy way to reproduce the problem is by pressing F6 from the editor to focus the status bar and use the arrow keys to move between elements. The issue only seems to affect NVDA at the moment. It may be good to give some time to fire these updates after the control receives focus, especially if the user is navigating quickly through nearby controls like in the status bar. In fact, I wonder what is the benefit of showing just the name of controls and potentially some descriptive text in the accessible view.

@isidorn @meganrogge

VS Code version: Code - Insiders 1.85.0-insider (94a3d7b, 2023-12-01T19:00:32.802Z)
OS version: Windows_NT arm64 10.0.22631
Modes:

System Info
Item Value
CPUs Apple Silicon (6 x 3200)
GPU Status 2d_canvas: enabled
canvas_oop_rasterization: disabled_off
direct_rendering_display_compositor: disabled_off_ok
gpu_compositing: enabled
multiple_raster_threads: enabled_on
opengl: enabled_on
rasterization: unavailable_off
raw_draw: disabled_off_ok
video_decode: enabled
video_encode: enabled
vulkan: disabled_off
webgl: enabled
webgl2: enabled
webgpu: enabled
Load (avg) undefined
Memory (System) 15.99GB (9.79GB free)
Process Argv --crash-reporter-id 286be978-64ae-4658-8bf9-a3235a9d2cd2
Screen Reader yes
VM 67%
A/B Experiments
vsliv695:30137379
vsins829:30139715
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythontb:30258533
pythonptprofiler:30281269
vshan820:30294714
vscod805cf:30301675
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
pythonvs932:30404738
py29gd2263:30784851
a247i228:30880122
vsclangdf:30492506
c4g48928:30535728
dsvsc012cf:30540253
showlangstatbar:30737417
fixshowwlkth:30771523
showindicator:30805243
pythongtdpath:30726887
i26e3531:30792625
welcomedialogc:30812479
pythonnosmt12:30779711
pythonidxpt:30768918
pythonnoceb:30776497
asynctok:30898717
dsvsc013:30777762
dsvsc014:30777825
pythonmhint1:30901775
dsvsc015:30821418
pythontestfixt:30866404
pythonregdiag2:30902439
pyreplss1:30879911
pythonmypyd1:30859725
pythoncet0:30859736
pythontbext0:30879054
accentitlementst:30870582
dsvsc016:30879898
dsvsc017:30880771
dsvsc018:30880772
aa_t_chat:30882232

@isidorn isidorn added the accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues label Dec 4, 2023
@meganrogge
Copy link
Contributor

Thanks for the report. It shouldn't be reading info about the accessible view in this case.

@meganrogge meganrogge added the bug Issue identified by VS Code Team member as probable bug label Dec 4, 2023
@meganrogge
Copy link
Contributor

@rperez030 last iteration, for keyboard only users, we added support for showing the tooltip on keyboard focus of status bar items. I had only tested this with a screen reader on macOS. I'll be curious to see why this is causing an issue on windows.

@rperez030
Copy link
Contributor Author

rperez030 commented Dec 4, 2023

I'm debugging with NVDA. Each time an element in the status bar gains focus, NVDA reports a liveRegionChange associated with the announcement "Inspect this in the accessible view with Alt+F2". If I press alt +F2, the name of that control appears in the accessible view.The live region update is taking precedence over the focus changed event.

The same announcement is present on the mac, but over their voiceover is prioritizing the focus event over the live region update, so the control that receives focus is announced first. The same happens with JAWS in windows.

@meganrogge meganrogge added the windows VS Code on Windows issues label Dec 4, 2023
meganrogge added a commit that referenced this issue Dec 4, 2023
@meganrogge meganrogge added this to the December 2023 milestone Dec 4, 2023
@VSCodeTriageBot VSCodeTriageBot added the unreleased Patch has not yet been released in VS Code Insiders label Dec 4, 2023
meganrogge added a commit that referenced this issue Dec 4, 2023
@meganrogge meganrogge reopened this Dec 4, 2023
@meganrogge meganrogge modified the milestones: December 2023, November 2023 Dec 4, 2023
@VSCodeTriageBot VSCodeTriageBot removed the unreleased Patch has not yet been released in VS Code Insiders label Dec 4, 2023
@meganrogge meganrogge added the candidate Issue identified as probable candidate for fixing in the next release label Dec 4, 2023
meganrogge added a commit that referenced this issue Dec 4, 2023
@rzhao271 rzhao271 closed this as completed Dec 4, 2023
@meganrogge
Copy link
Contributor

To verify, on windows, enable NVDA. Focus the status bar / activity bar. Use arrow keys. Verify that there's no message about opening the accessible view. Focus a hover in the editor using only the keyboard. Verify that you do hear a message about opening the accessible view.

@meganrogge meganrogge added verified Verification succeeded and removed verified Verification succeeded labels Dec 5, 2023
@meganrogge
Copy link
Contributor

This was verified in insider's by @rperez030. Will leave it without the label to also verify in the stable build.

@rzhao271 rzhao271 added the verified Verification succeeded label Dec 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues bug Issue identified by VS Code Team member as probable bug candidate Issue identified as probable candidate for fixing in the next release verified Verification succeeded windows VS Code on Windows issues
Projects
None yet
5 participants