Restore the TinyMCE Help modal keyboard shortcut for the Classic Editor plugin

[afercia]

[44257] removed the keyboard shortcut to open the TinyMCE help modal. In Gutenberg, this shortcut now opens the "global" help modal.

However, when the Classic Editor plugin is activated, the tooltip still displays the keyboard shortcut even if the shortcut doesn't do anything:

Instead of removing the displayed shortcut from the tooltip, I'd propose to restore it only when Classic Editor is used, much like what was done for the access+z shortcut, see https://core.trac.wordpress.org/browser/trunk/src/js/_enqueues/vendor/tinymce/plugins/wordpress/plugin.js?rev=44257#L670. This wouldn't affect anything in Gutenberg and seems pretty easy to implement.

Also, worth considering when ​https://github.com/WordPress/gutenberg/issues/6195 will be addressed, the shortcut will be displayed also in the tooltip in the Gutenberg Classic block. However, it now opens the "global" help modal. Should the passed string check for the editor type and display the shortcut conditionally?

[afercia]

See also #39271 which mentions all the shortcuts that use Control+Option+something.

[afercia]

Discussed during today's accessibility bug scrub. Agreed to move this ticket to 5.1.x and assign JB as owner.

[azaozz]

[44257] removed the keyboard shortcut to open the TinyMCE help modal
...
However, when the Classic Editor plugin is activated, the tooltip still displays the keyboard shortcut even if the shortcut doesn't do anything

That's not (shouldn't be) the case? :)

The shortcut (should) work as always in the Classic Editor and the Classic Block (when focused). [44257] prevents that shortcut from triggering both help modals at the same time in the Block Editor.

[afercia]

Yeah I understand the purpose of [44257] but with the Classic Editor plugin activated, the shortcut doesn't work :)

Also to note: the iframe title still instructs screen reader users to use that shortcut: Rich Text Area. Press Control-Option-H for help.

[azaozz]

Replying to afercia:

Hmmm, seems to work here in both Classic Block and Classic Editor, Alt+Shift+H on Win10. Tested in Firefox, Chrome and Edge.

The iframe tag in Classic Editor is also correct as far as I see:

<iframe id="content_ifr" frameborder="0" allowtransparency="true" title="Rich Text Area. Press Alt-Shift-H for help." style="width: 100%; height: 300px; display: block;"></iframe>

Or perhaps I'm misunderstanding where to look? :)

[afercia]

Ah then it must be macOS specific? Interestingly, I've just noticed:

Control Option H doesn't work

Shift Option H does work. However, it is supposed to be Control-Option-H.

The iframe title is fine, was just saying that it's misleading as it creates the expectation the shortcut is available, while it actually isn't.

[azaozz]

Uh, I see where that comes from. It only tests for event.shiftKey && event.altKey (which on Mac is used for typing accent characters, etc.). Should look at the OS first.

[azaozz]

In 44913:

TinyMCE: fix the TinyMCE help modal keyboard shortcut on Mac for the classic editor and classic block.

Fixes #45791.