microsoft / vscode

Visual Studio Code
https://code.visualstudio.com
MIT License
161.63k stars 28.4k forks source link

Voice over is announcing as incorrect information that "you are in a tree grid" for "Call Stack section" which has no data inside it: A11y_Visual Studio Code Client_Sidebar Bar Options_Accessing Debug sidebar options_Screen Reader. #221563

Closed keerthiduvvuri closed 1 month ago

keerthiduvvuri commented 1 month ago

GIT Hub Tags:

AILimited; #A11yTCS; #A11ySev2; #Visual Studio Code Client; #BM_Visual Studio Code Client_MAC_JULY2024; #DesktopApp; #WCAG1.3.1; #FTP; #A11yMAS; #Screen Reader; #Voiceover; #MAC; #Device;

Environment and OS details:

MAC Book Pro MAC OS Sonoma Version 14.5 Visual Studio code Insider version 1.92.0

Reproduction Steps:

  1. Open Visual Studio Code Insiders editor
  2. Using "Shift+Command+D -1" key navigate to the debug side view bar option.
  3. Using "Tab" key navigate to Debug side bar options.

Actual behavior:

Voice over is announcing as incorrect information that "you are in a tree grid" for "Call Stack section" which has no data inside it.

Expected behavior:

Voice over should not be announcing as incorrect information that "you are in a tree grid" for "Call Stack section" which has no data inside it.

User Impact:

Screen reader users will face difficulty if Voice over is announcing as incorrect information that "you are in a tree grid" for "Call Stack section" which has no data inside it.

Attachments:

https://github.com/user-attachments/assets/fbfa3c27-e425-4b68-969d-a3024b6a32b0

meganrogge commented 1 month ago

I am not sure that we should change the role depending on the state. Users can explore the tree grid and find there are no rows in there. @jooyoungseo and @rperez030 what do you think?

meganrogge commented 1 month ago

@keerthiduvvuri can you cite documentation that says the role should change depending on whether or not there's data in the element?

rperez030 commented 1 month ago

@meganrogge I totally agree with you. Unless I'm missing something, it is empty simply because there is no code running at that point. Also, @keerthiduvvuri can you please explain the rational for considering this a sev2? How is the user blocked in this case?

until we have more information, I would consider this an invalid bug.

PoojaNamde commented 1 month ago

Closed;

Closing this bug as not blocking issue.

meganrogge commented 1 month ago

Thanks, can this be closed now?

keerthiduvvuri commented 1 month ago

As per above comment closing this bug as invalid.