monaco-editor icon indicating copy to clipboard operation
monaco-editor copied to clipboard

[Bug][a11y] Accessibility help dialog is not displayed when alt + f1 is pressed > v0.40.0

Open jjaeggli opened this issue 2 years ago • 7 comments

Reproducible in vscode.dev or in VS Code Desktop?

  • [X] Not reproducible in vscode.dev or VS Code Desktop

Reproducible in the monaco editor playground?

Monaco Editor Playground Link

Recent versions of the Monaco editor do not display the accessibility help dialog when alt + f1 is pressed. The editor still announces the availability of the dialog when the text area of the editor is focused by a screen reader user. The help dialog is important for surfacing accessibility help. A new version of the dialog is displayed when using vscode desktop or OSS / web.

Playground Link:

https://microsoft.github.io/monaco-editor/playground.html?source=v0.40.0#example-creating-the-editor-hello-world

Monaco Editor Playground Code

https://microsoft.github.io/monaco-editor/playground.html?source=v0.40.0#example-creating-the-editor-hello-world

Reproduction Steps

  1. Open the monaco demo page for v0.40 and above
  2. Focus the editor and press alt + f1
  3. Observe that nothing happens

Actual (Problematic) Behavior

No accessibility help dialog is displayed

Expected Behavior

The accessibility help dialog is displayed

Additional Context

No response

jjaeggli avatar Aug 10 '23 17:08 jjaeggli

It looks like this is first introduced in v0.40.0-dev.20230609 https://microsoft.github.io/monaco-editor/playground.html?source=v0.40.0-dev.20230609#example-creating-the-editor-hello-world

jjaeggli avatar Aug 10 '23 17:08 jjaeggli

@isidorn do you know why the behavior here would be different in the monaco editor playground?

meganrogge avatar Aug 10 '23 20:08 meganrogge

Inspecting the runtime code with the debugger, it looks like editor.action.showAccessibilityHelp is a registered command in v0.39 but in v0.40 it is not.

jjaeggli avatar Aug 10 '23 21:08 jjaeggli

Thanks for debugging. That explains it. The command is called editor.action.accessibilityHelp now.

meganrogge avatar Aug 10 '23 21:08 meganrogge

@jjaeggli thanks for filling this issue and for finding the culprit it seems. Assigning to @hediet and @meganrogge

isidorn avatar Aug 11 '23 07:08 isidorn

@hediet is there a way to bring this fix to v0.39.0? IE updating the command name

meganrogge avatar Aug 11 '23 15:08 meganrogge

@hediet @meganrogge any updates on the following this issue?

zeus2198 avatar Apr 15 '24 12:04 zeus2198