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

Voiceover when expanding selection could be better #144791

Closed
JacksonKearl opened this issue Mar 9, 2022 · 12 comments · Fixed by #166430
Closed

Voiceover when expanding selection could be better #144791

JacksonKearl opened this issue Mar 9, 2022 · 12 comments · Fixed by #166430
Assignees
Labels
accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues bug Issue identified by VS Code Team member as probable bug chromium Issues and items related to Chromium fixed-in-electron-22 Issues fixed with Electron 22.x update macos Issues with VS Code on MAC/OS X upstream Issue identified as 'upstream' component related (exists outside of VS Code) upstream-issue-fixed The underlying upstream issue has been fixed verified Verification succeeded
Milestone

Comments

@JacksonKearl
Copy link
Contributor

Couple issues:

  1. when expanding the ***** selection, it only ever says 1star star star` selected even when more *'s are shown in the text. This seems to be a macos global bug
  2. The formatting foo unselected, foo bar selected is not great, it hides the most important info. This seems to be a chromium thing, repros in edge, but notes handles it better IMO

Recording 2022-03-09 at 14 14 46

Recording 2022-03-09 at 14 20 03

@JacksonKearl JacksonKearl added the accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues label Mar 9, 2022
@isidorn
Copy link
Contributor

isidorn commented Mar 10, 2022

@JacksonKearl can you reproduce with VoiceOver and textArea here https://developer.mozilla.org/en-US/docs/Web/HTML/Element/textarea
If yes, can you please file a new issue for VoiceOver here https://bugs.webkit.org/
Then I can share the bug with the VoiceOver authors. Thanks!

@JacksonKearl
Copy link
Contributor Author

It works fine in WebKit, seems to be only Chromium/Electron that's effected.

@isidorn
Copy link
Contributor

isidorn commented Apr 6, 2022

@JacksonKearl if you can reproduce with Chromium then can you file a new issue in https://bugs.chromium.org/p/chromium/issues/list

Or are you saying you can only reproduce with Electron?

@JacksonKearl
Copy link
Contributor Author

Sure, reported here https://bugs.chromium.org/p/chromium/issues/detail?id=1313990 👍

@isidorn isidorn added upstream Issue identified as 'upstream' component related (exists outside of VS Code) chromium Issues and items related to Chromium labels Apr 7, 2022
@isidorn isidorn added this to the Backlog milestone Apr 7, 2022
@MarcoZehe
Copy link
Contributor

Is #162152 a duplicate of this bug?

@JacksonKearl
Copy link
Contributor Author

@MarcoZehe yes I believe so, good news is chromium fixed this 12 days ago! https://bugs.chromium.org/p/chromium/issues/detail?id=1172956#c7

Bad news is it may be a while before vscode gets that build.

@MarcoZehe
Copy link
Contributor

Thanks! Yes, I saw that it is fixed now, and I also discovered that I was the one who originally filed this bug report in February 2021. I couldn't figure out what version of Chromium this will be in, though, but I suppose it is currently in Canary and has to move through beta and release for the next couple of weeks, then to Electron, until it finally hits VSCode once it moves to that version of Electron that has the fix.

@isidorn
Copy link
Contributor

isidorn commented Nov 28, 2022

@MarcoZehe thanks for opening the original issue on the Chromium side.
@deepak1556 can we back port the chromium fix so it is included as part of our next Electron update?
@meganrogge fyi

@JacksonKearl hey there 👋

@meganrogge meganrogge self-assigned this Nov 28, 2022
@deepak1556
Copy link
Collaborator

Fixed upstream issue: https://bugs.chromium.org/p/chromium/issues/detail?id=1172956

I will backport the fix with Electron 22 update.

@deepak1556 deepak1556 removed this from the Backlog milestone Dec 8, 2022
@deepak1556 deepak1556 added bug Issue identified by VS Code Team member as probable bug upstream-issue-fixed The underlying upstream issue has been fixed fixed-in-electron-22 Issues fixed with Electron 22.x update labels Dec 8, 2022
@deepak1556 deepak1556 self-assigned this Dec 8, 2022
@deepak1556
Copy link
Collaborator

Reopening because Electron 22 was reverted for the March milestone.

@deepak1556 deepak1556 reopened this Mar 22, 2023
@deepak1556 deepak1556 added this to the April 2023 milestone Mar 22, 2023
@alexr00
Copy link
Member

alexr00 commented Apr 27, 2023

I can't reproduce the original issue in stable. I don't get the "__ unselected" when expanding the selection. Is this a macOS specific bug?

@alexr00 alexr00 added the verification-steps-needed Steps to verify are needed for verification label Apr 27, 2023
@MarcoZehe
Copy link
Contributor

MarcoZehe commented Apr 27, 2023 via email

@alexr00 alexr00 added macos Issues with VS Code on MAC/OS X and removed verification-steps-needed Steps to verify are needed for verification labels Apr 28, 2023
@TylerLeonhardt TylerLeonhardt added the verified Verification succeeded label Apr 28, 2023
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 bug Issue identified by VS Code Team member as probable bug chromium Issues and items related to Chromium fixed-in-electron-22 Issues fixed with Electron 22.x update macos Issues with VS Code on MAC/OS X upstream Issue identified as 'upstream' component related (exists outside of VS Code) upstream-issue-fixed The underlying upstream issue has been fixed verified Verification succeeded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants