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

Consider to set an aria label for each quick pick when it becomes active #93947

Closed
weinand opened this issue Mar 31, 2020 · 5 comments
Closed
Assignees
Labels
accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues bug Issue identified by VS Code Team member as probable bug quick-pick Quick-pick widget issues verified Verification succeeded
Milestone

Comments

@weinand
Copy link
Contributor

weinand commented Mar 31, 2020

Refs #93648
Orca 3.28.0 on Ubuntu 18.04.4 LTS

  • open a quickpick (e.g. file quickpick or command quickpick)

Screen reader says something like "type to narrow down ??? combo box" (I couldn't understand the word "???").
Instead of saying "combo box" quickpick could be more specific and say "files" or "commands".

[I have no experience with screen readers. I just report what I noticed.
Someone who has to rely on a screen reader should test this and provide feedback.]

@bpasero bpasero added accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues quick-pick Quick-pick widget issues labels Mar 31, 2020
@bpasero bpasero added this to the March 2020 milestone Mar 31, 2020
@chrmarti
Copy link
Contributor

chrmarti commented Apr 1, 2020

We could add an aria label that each use of the QuickPick would need to supply. Alternatively we typically have a placeholder that is context specific (but might not always speak for itself from what I see).

@bpasero
Copy link
Member

bpasero commented Apr 1, 2020

I am using placeholder exactly for that reason and was under the assumption that screen readers would read it out to the user.

@chrmarti for the aria label I think this needs infrastructure on the level of quick pick, I only see the ariaLabel attribute for items.

Moving to April, would also like to hear @isidorn input on this and potentially real screen reader users.

@bpasero bpasero modified the milestones: March 2020, April 2020 Apr 1, 2020
@bpasero bpasero added the under-discussion Issue is under discussion for relevance, priority, approach label Apr 1, 2020
@bpasero bpasero changed the title Screenreader could be more specific when quickpick opens Consider to set an aria label for each quick pick when it becomes active Apr 1, 2020
@isidorn
Copy link
Contributor

isidorn commented Apr 1, 2020

The placeholder should be read. Absolutely agree on that.
I now tried it with VoiceOver and the placeholder does not get read.
@bpasero I see that you set the ariaLabel to the "Type to narrow down results". Why not simple set it to the placeholder message?

@MarcoZehe @jvesouza @ajborka feedback very welcome on what should be read when the quick pick gets opened (for example via cltr/cmd + p)

Screenshot 2020-04-01 at 13 55 36

@bpasero bpasero added bug Issue identified by VS Code Team member as probable bug and removed under-discussion Issue is under discussion for relevance, priority, approach labels Apr 1, 2020
@bpasero bpasero closed this as completed in 81278ba Apr 1, 2020
@bpasero bpasero modified the milestones: April 2020, March 2020 Apr 1, 2020
@weinand weinand added the verified Verification succeeded label Apr 2, 2020
@weinand
Copy link
Contributor Author

weinand commented Apr 2, 2020

With "F1" screen reader now says:
"Leaving main content. Type the name of a command to run editable combo box. Opens menu".

For "Cmd-P" screen reader reads the placeholder text (which is a bit long but gets the message across).

@jvesouza
Copy link

jvesouza commented Apr 2, 2020

LOL! It was 'Show de bola' as we say here in Brazil, when something is very good.

When I press ctrl + p, orca announces the following:
Search files by name (append : to go to line or @ to go to symbol)

I think I'm falling in love with VSCode. I hope my wife doesn't read this since she doesn't know what VSCode is.

@github-actions github-actions bot locked and limited conversation to collaborators May 16, 2020
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 bug Issue identified by VS Code Team member as probable bug quick-pick Quick-pick widget issues verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

5 participants