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

Type of suggestion not indicated to screen reader users #239520

Closed
meganrogge opened this issue Feb 3, 2025 · 11 comments · Fixed by #243241 or #244761
Closed

Type of suggestion not indicated to screen reader users #239520

meganrogge opened this issue Feb 3, 2025 · 11 comments · Fixed by #243241 or #244761
Assignees
Labels
accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues feature-request Request for new features or functionality insiders-released Patch has been released in VS Code Insiders on-release-notes Issue/pull request mentioned in release notes suggest IntelliSense, Auto Complete verification-needed Verification of issue is requested verified Verification succeeded
Milestone

Comments

@meganrogge
Copy link
Contributor

When using ctrl+space, to get suggestions, sighted users have an icon which tells them the type of suggestion that is currently focused - a symbol, function, method, etc. We should indicate this information to screen reader users.

I am thinking we should add this information to the end of the aria label, but also don't want it to be too verbose.

cc @jooyoungseo, @rperez030 for feedback

@meganrogge meganrogge self-assigned this Feb 3, 2025
@meganrogge meganrogge added feature-request Request for new features or functionality accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues suggest IntelliSense, Auto Complete labels Feb 3, 2025
@rperez030
Copy link
Contributor

I think this could follow the same convention as the "Go to Symbol in Editor" command.

@meganrogge
Copy link
Contributor Author

For me, on macOS with VoiceOver, I do not hear any symbol icon info when using Go To Symbol in Editor. @rperez030 what do you hear?

@rperez030
Copy link
Contributor

For me, on macOS with VoiceOver, I do not hear any symbol icon info when using Go To Symbol in Editor. @rperez030 what do you hear?

You are right. It sounds like we have another issue to investigate. on Windows i hear something like: "encode_image (function) 2 of 10", but Voiceover only reads the symbol. another place where we use the same convention is the outline view inside Explorer (Explorer: Focus on Outline view command). This one work well in both platforms.

@meganrogge meganrogge added this to the March 2025 milestone Feb 24, 2025
meganrogge added a commit that referenced this issue Mar 11, 2025
@vs-code-engineering vs-code-engineering bot added unreleased Patch has not yet been released in VS Code Insiders insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels Mar 12, 2025
@meganrogge meganrogge added the verification-needed Verification of issue is requested label Mar 24, 2025
@meganrogge
Copy link
Contributor Author

To verify:

  • start a screen reader
  • in the editor and terminal separately, invoke suggestions
  • verify that you hear the type of suggestion at the end of the aria label

@roblourens
Copy link
Member

I don't think I hear the suggestion type anywhere, what would that be like? Tried some different types in editor and terminal

Image

@roblourens roblourens reopened this Mar 25, 2025
@roblourens roblourens added the verification-found Issue verification failed label Mar 25, 2025
@vs-code-engineering vs-code-engineering bot removed the insiders-released Patch has been released in VS Code Insiders label Mar 25, 2025
@jrieken jrieken removed their assignment Mar 26, 2025
@meganrogge
Copy link
Contributor Author

meganrogge commented Mar 26, 2025

I can reproduce 🤔 . I see that the aria label is correctly set in both places

Image Image

@meganrogge
Copy link
Contributor Author

I also see it gets applied here
Image

@meganrogge
Copy link
Contributor Author

I think that role of option is the issue here - when that's the case, screen readers pay attention to the text of the option instead of the aria label.

@meganrogge
Copy link
Contributor Author

meganrogge commented Mar 26, 2025

listitem works

@meganrogge
Copy link
Contributor Author

@rperez030 it would be great if you could pls test this fix in insider's tomorrow. Use ctrl+space in the editor to try it out.

@vs-code-engineering vs-code-engineering bot added unreleased Patch has not yet been released in VS Code Insiders insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels Mar 26, 2025
@meganrogge meganrogge added the verified Verification succeeded label Mar 27, 2025
@meganrogge
Copy link
Contributor Author

This was tested by Roberto

@meganrogge meganrogge added the on-release-notes Issue/pull request mentioned in release notes label Mar 28, 2025
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 feature-request Request for new features or functionality insiders-released Patch has been released in VS Code Insiders on-release-notes Issue/pull request mentioned in release notes suggest IntelliSense, Auto Complete verification-needed Verification of issue is requested verified Verification succeeded
Projects
None yet
4 participants