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

prevent screen reader from reading a user's chat request on enter #185153

Merged
merged 5 commits into from
Jun 15, 2023

Conversation

meganrogge
Copy link
Contributor

@meganrogge meganrogge commented Jun 14, 2023

fixes https://github.com/microsoft/vscode-internalbacklog/issues/4181

cc @jooyoungseo, @rperez030

Since a user has just typed their input, we don't want a screen reader to read it when enter is pressed. This is also important to fix as it blocks the progress audio cue that we will have as an option.

Unblocks https://github.com/microsoft/vscode-copilot/issues/197

@meganrogge meganrogge self-assigned this Jun 14, 2023
@meganrogge meganrogge added this to the June 2023 milestone Jun 14, 2023
@meganrogge
Copy link
Contributor Author

I tested on Windows and discussed this with @rperez030

Copy link
Member

@roblourens roblourens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm ok with this but I would appreciate some comments because otherwise it's basically WTF code haha

@meganrogge meganrogge merged commit 0987474 into main Jun 15, 2023
6 checks passed
@meganrogge meganrogge deleted the merogge/dupe-speech branch June 15, 2023 16:22
@github-actions github-actions bot locked and limited conversation to collaborators Jul 30, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants