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

QuickOpen: add aria-setsize and aria-posinset #76709

Closed
bhagwatshelke1047 opened this issue Jul 5, 2019 · 8 comments
Closed

QuickOpen: add aria-setsize and aria-posinset #76709

bhagwatshelke1047 opened this issue Jul 5, 2019 · 8 comments
Assignees
Labels
accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues *duplicate Issue identified as a duplicate of another issue(s) feature-request Request for new features or functionality quick-pick Quick-pick widget issues

Comments

@bhagwatshelke1047
Copy link

Environment Details:
VSCode Version : 1.36.1 (user setup)
OS Version : Win10

Additional Details:
MAS Violated : MAS 1.3.1
Screen Reader Versions : JAWS 2019

Repro Steps:
1.Open Visual studio code editor.
2.Open the command Pallet by pressing "F1" key and type "Theme"
3.Using Down arrow key select "Preference colour Theme" and press enter key to select.
4.Using "Up/Down" arrow key select different available theme.
5.Start screen reader.

Actual:
Screen reader does not announce the ordering that is 'light 1 of 15' for drop down item of 'select colour theme' combo box.

This issue is observed in Win10 + JAWS

Expected:
Screen reader should announce the proper ordering that is 'light 1 of 15' for each drop down item of 'select colour theme' combo box.

Recommendations:
Refer below link which is repository of bug fixes code snippets:
https://microsoft.sharepoint.com/teams/msenable/mas/pages/browse-fixes.aspx

User Impact:
Screen reader users will not get the exact information.

MAS Reference:
MAS 1.3.1 - https://microsoft.sharepoint.com/:w:/r/teams/msenable/_layouts/15/WopiFrame.aspx?sourcedoc={54f28d1f-a2d1-4dcd-84e1-5c9b87e8aba4}

Attachment for Reference:
A11y_VSCode_ChangingoftheTheme_ScreenReader.pptx

@vscodebot vscodebot bot added the themes Color theme issues label Jul 5, 2019
@aeschli
Copy link
Contributor

aeschli commented Jul 8, 2019

@isidorn I'm using the quick pick service for that and I think this should be added there (or we need to add API so that I can set the labels that are read out)

@isidorn
Copy link
Contributor

isidorn commented Jul 8, 2019

Currently we do not support JAWS, only NVDA on windows and VoiceOver on mac. More about this can be found here https://code.visualstudio.com/docs/editor/accessibility#_screen-readers

And this works just fine with VoiceOver thus closing.
Please let me know if you can reproduce this with NVDA and I will reopen

lala

@isidorn isidorn closed this as completed Jul 8, 2019
@isidorn isidorn added invalid Issue identified as not relevant or not valid accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues labels Jul 8, 2019
@isidorn
Copy link
Contributor

isidorn commented Jul 8, 2019

However I do agree we should read the order. Due to that reopening (since I originally misunderstood the issue).
For this we need to add aria-setsize and aria-posinset attributes.

@bpasero can you provide some code pointers for quick open since @chrmarti is out? So I try to add these attributes?

@isidorn isidorn reopened this Jul 8, 2019
@isidorn isidorn changed the title A11y_VSCode_ChangingoftheTheme_ScreenReader: Screen reader does not announce the ordering of items present in the dropdown list of 'select color theme' combo box. QuickOpen: add aria-setsize and aria-posinset Jul 8, 2019
@isidorn isidorn added feature-request Request for new features or functionality quick-pick Quick-pick widget issues and removed invalid Issue identified as not relevant or not valid themes Color theme issues labels Jul 8, 2019
@isidorn isidorn assigned chrmarti and unassigned aeschli Jul 8, 2019
@bpasero
Copy link
Member

bpasero commented Jul 8, 2019

@isidorn this is not quick open related is it? these attributes should be on the tree/list layer, no?

@isidorn
Copy link
Contributor

isidorn commented Jul 8, 2019

They are already on the list layer. But the quick open is using the old tree.
Thus duplicate of #69955

@isidorn isidorn closed this as completed Jul 8, 2019
@isidorn isidorn added the *duplicate Issue identified as a duplicate of another issue(s) label Jul 8, 2019
@bpasero
Copy link
Member

bpasero commented Jul 8, 2019

@isidorn not sure, all pickers use the new tree and I think this is about the theme picker?

@isidorn
Copy link
Contributor

isidorn commented Jul 8, 2019

@bpasero you are correct. We are using the new tree and the aria-setsize and aria-posinset are set.
This all works fine on NVDA and voice over. The issue here is that the tester is using JAWS, which we do not support. More details about this can be found here (we support NVDA) https://code.visualstudio.com/docs/editor/accessibility.

Screenshot 2019-07-08 at 14 08 02

@vscodebot vscodebot bot locked and limited conversation to collaborators Aug 22, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues *duplicate Issue identified as a duplicate of another issue(s) feature-request Request for new features or functionality quick-pick Quick-pick widget issues
Projects
None yet
Development

No branches or pull requests

5 participants