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

Notebook cell editor 'is not accessible at this time' #183967

Closed
joyceerhl opened this issue May 31, 2023 · 5 comments
Closed

Notebook cell editor 'is not accessible at this time' #183967

joyceerhl opened this issue May 31, 2023 · 5 comments
Assignees

Comments

@joyceerhl
Copy link
Collaborator

While testing #180470

  1. Enable screenreader
  2. Click into notebook cell
  3. 🐛 hear 'the editor is not accessible at this time'
@meganrogge
Copy link
Contributor

meganrogge commented May 31, 2023

That's coming from here. @joyceerhl were you in Screen reader optimized mode?

return '';
}
private _getAriaLabel(options: IComputedEditorOptions): string {
const accessibilitySupport = options.get(EditorOption.accessibilitySupport);
if (accessibilitySupport === AccessibilitySupport.Disabled) {
return nls.localize('accessibilityOffAriaLabel', "The editor is not accessible at this time. Press {0} for options.", platform.isLinux ? 'Shift+Alt+F1' : 'Alt+F1');
}
return options.get(EditorOption.ariaLabel);
}
private _setAccessibilityOptions(options: IComputedEditorOptions): void {

@joyceerhl
Copy link
Collaborator Author

No I just had my screen reader turned on from outside VS Code, and VS Code didn't ask me to enable screen reader optimized mode (although all the verbosity hints were picked up correctly)

@meganrogge
Copy link
Contributor

Then the root of this bug is we're not detecting your screen reader correctly cc @deepak1556

Which were you using? NVDA, JAWS, Narrator?

@meganrogge meganrogge assigned deepak1556 and unassigned rebornix May 31, 2023
@joyceerhl
Copy link
Collaborator Author

joyceerhl commented May 31, 2023

After digging around in settings I think the problem is that I'd set "editor.accessibilitySupport" to "off" some time ago, so even though I had a screen reader attached, editor accessibility support wasn't enabled--but all the accessibility verbosity settings were still functional, so I didn't initially realize that I'd disabled editor accessibility support. Is there a real end-user scenario for turning accessibility support off?

@meganrogge
Copy link
Contributor

Thanks for investigating. There is no benefit to turning accessibility support off when using a screen reader. So I think this can be closed unless you have a suggestion for different behavior @joyceerhl

@joyceerhl joyceerhl closed this as not planned Won't fix, can't repro, duplicate, stale Jun 1, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Jul 16, 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

No branches or pull requests

4 participants