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

the dialog that is displayed after checking for updates fails to respect Orca's silence speech feature #93056

Closed
ajborka opened this issue Mar 19, 2020 · 16 comments
Assignees
Labels
accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues chromium Issues and items related to Chromium linux Issues with VS Code on Linux upstream-issue-linked This is an upstream issue that has been reported upstream

Comments

@ajborka
Copy link

ajborka commented Mar 19, 2020

In vscode insiders 1.44, the dialog that is displayed after checking for updates fails to respect Orca's silence speech feature. This makes navigating the dialog difficult at best.

  1. Start Orca
  2. Start vscode insiders.
  3. In the manage toolbar of the main window, press the manage button
  4. From the popup menu, choose 'check for updates...'

Actual results: Orca reads the entire contents of the dialog even if the user presses the ctrl key to silence speech. Any actions taken while Orca reads the dialog contents are not spoken until Orca finishes the previous block of text.
Expected result: Interupting Orca's speech should stop Orca from reading the remaining contents of the dialog.

@vscodebot
Copy link

vscodebot bot commented Mar 19, 2020

(Experimental duplicate detection)
Thanks for submitting this issue. Please also check if it is already covered by an existing one, like:

@isidorn
Copy link
Contributor

isidorn commented Mar 20, 2020

@ajborka is the dialog you are getting "There are currently no updates available"? Can you also reproduce if you do Help > about vs code?
Because I can.

And this smells like we use aria.alert for dialogs and it feels like the Orca ctrl does not silence alerts. @joanmarie is Orca slience designed in such a way?

@deepak1556 also assinging also to you since this is a native dialog and might be somewhat connected to, since if we had nice reading of dialog content we would not have to use alerts
electron/electron#14234

@isidorn isidorn added accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues under-discussion Issue is under discussion for relevance, priority, approach linux Issues with VS Code on Linux labels Mar 20, 2020
@ajborka
Copy link
Author

ajborka commented Mar 20, 2020

@isidorn yes, it happens when getting the no updates available dialog, the help>about dialog, and I suspect the open/save/save as... dialogs might have the same problem, although the lag isn't as bad. Do you want different issues for the open/save/save as.../the 'do you want to save?' dialogs? because they all produce the same problem to some degree.

@joanmarie
Copy link

Lemme take a look.

@joanmarie
Copy link

joanmarie commented Mar 20, 2020

Ok, that's this issue https://bugs.chromium.org/p/chromium/issues/detail?id=1042864

Orca interrupts speech when keys are pressed; not just Ctrl. So you can also see Orca failing to shut up in the About dialog for VSCode as you Tab between its buttons.

The way Orca does this speech interruption is by getting a notification that a key was pressed and then doing the interruption. If Orca isn't told the user pressed a key, it doesn't know it needs to shut up. Sorry! Hopefully one of my colleagues will fix that Chromium issue soon.

@deepak1556 deepak1556 added chromium Issues and items related to Chromium upstream-issue-linked This is an upstream issue that has been reported upstream labels Mar 20, 2020
@isidorn isidorn removed the under-discussion Issue is under discussion for relevance, priority, approach label Mar 23, 2020
@isidorn
Copy link
Contributor

isidorn commented Mar 23, 2020

@joanmarie Thanks a lot for linking to the chromum issue.

@isidorn isidorn added this to the Backlog milestone Jun 9, 2020
@isidorn
Copy link
Contributor

isidorn commented Aug 14, 2020

@deepak1556 can you check if the issue https://bugs.chromium.org/p/chromium/issues/detail?id=1042864 is part of the latest Electron which we have and then maybe we should add a "fixed-in-electron-9" label here?

Also can someone teach me how to figure out what chrome version a fix ends up in? Is that specified somewhere?

@alex19EP
Copy link

alex19EP commented Sep 3, 2020

@deepak1556 can you check if the issue https://bugs.chromium.org/p/chromium/issues/detail?id=1042864 is part of the latest Electron which we have and then maybe we should add a "fixed-in-electron-9" label here?

Also can someone teach me how to figure out what chrome version a fix ends up in? Is that specified somewhere?

i am not @deepak1556 but I can say for sure that the problem is not fixed in electron 9.

as for chromium version, I'd like to know that too. 🙂

@isidorn
Copy link
Contributor

isidorn commented Sep 3, 2020

Thanks. Then Electron 10 maybe :)

@deepak1556
Copy link
Collaborator

The fix landed first in 86.0.4199.0, which would be Electron 11. I will look into backporting to 10 and 9 for next iteration.

@deepak1556 deepak1556 modified the milestones: Backlog, September 2020 Sep 3, 2020
@isidorn
Copy link
Contributor

isidorn commented Sep 4, 2020

Great, thanks. Yeah if the backport is not too complex it would be great to have this.

@isidorn
Copy link
Contributor

isidorn commented Oct 1, 2020

@deepak1556 we should move this to October?

@deepak1556
Copy link
Collaborator

Exploration builds based on Electron 11:

Zip Archive
Deb
Snap
RPM

@isidorn
Copy link
Contributor

isidorn commented Oct 27, 2020

Added fixed in electron 11 label, since @jvesouza confirmed this is fixed with exploration builds.

@webczat
Copy link

webczat commented Oct 28, 2020

finally! I know chromium unstable has this fixed and I admit I was wondering when this would land here. 👍

@ajborka
Copy link
Author

ajborka commented Oct 28, 2020

@deepak1556 , @isidorn this appears to be fixed. Nice job.

@github-actions github-actions bot locked and limited conversation to collaborators Mar 11, 2021
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 chromium Issues and items related to Chromium linux Issues with VS Code on Linux upstream-issue-linked This is an upstream issue that has been reported upstream
Projects
None yet
Development

No branches or pull requests

6 participants