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

Keyboard shortcuts that use the access modifier don't work with VoiceOver #11154

Open
afercia opened this issue Oct 27, 2018 · 5 comments
Open
Labels
[Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). [Type] Bug An existing feature does not function as intended

Comments

@afercia
Copy link
Contributor

afercia commented Oct 27, 2018

Splitting this out from #5694 see #10545

Gutenberg uses a convention for keyboard shortcuts where the access combination translates, on macOS, to Control Option. Currently, there are a few shortcuts that use this modifier, including the new Blocks navigation menu implemented in #10545.

Unfortunately, VoiceOver on macOS uses Control Option as its own default modifier so all the Gutenberg shortcuts that use Control Option either trigger a specific VoiceOver feature or don't work at all.

So far, I've found the following shortcuts using Control Option:

Display this help. ^ ⌥ H
Open the block navigation menu. ^ ⌥ O
Remove the selected block(s). ^ ⌥ Z
Remove a link. ^ ⌥ S
Add a strikethrough to the selected text. ^ ⌥ D
Display the selected text in a monospaced font. ^ ⌥ X
Add link ^ ⌥ a (undocumented in the Help Modal, alternative shortcut available)

Basically, all the shortcuts implemented with rawShortcut.access() or <Shortcut type="access" .... All of these need to be changed to use a different modifier.

Aside: still thinking the best path forward is a mechanism to allow users to re-map and customize shortcuts, as proposed in #3218.

@afercia afercia added [Type] Bug An existing feature does not function as intended [Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). labels Oct 27, 2018
@ellatrix
Copy link
Member

So this is also an issue with the old editor? WordPress has used the "access" modifier since a long time, and it was hard to find anything that didn't clash with the OS or browser. Has this always been the same in VoiceOver?

@afercia
Copy link
Contributor Author

afercia commented Oct 31, 2018

Correct.
For example, testing with Safari + VoiceOver, Ctrl-Alt-{1-6} to change a heading level doesn't work.

@afercia
Copy link
Contributor Author

afercia commented Jan 4, 2019

See related core Trac ticket: https://core.trac.wordpress.org/ticket/39271

@afercia
Copy link
Contributor Author

afercia commented Mar 7, 2019

For reference: in the new WAI-ARIA Authoring Practices 1.1
there's a pretty good section dedicated to keyboard shortcuts. It's more about what not to do:
https://www.w3.org/TR/wai-aria-practices-1.1/#kbd_shortcuts_assigning

Among others, avoid:

macOS only: Control+Option + any combination of other keys.

@tbdalgaard
Copy link

Most of the issue here could be fixed if option was removed so the keystroke would be ctrl followed by the letter/number instead. That would solve the issues on Mac OS with Safari.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

No branches or pull requests

4 participants