Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#27024 closed defect (bug) (fixed)

Expose keyboard shortcut in Help documentation for TinyMCE

Reported by: joedolson's profile joedolson Owned by: nacin's profile nacin
Milestone: 3.9 Priority: normal
Severity: normal Version: 3.9
Component: Help/About Keywords: has-patch
Focuses: accessibility, docs Cc:

Description

In the post editor, you can shift from the editor into the TinyMCE controls by pressing Alt+F10. This is exposed to screen readers when they enter the post toolbar, but is not available at all to users who are keyboard dependent but not using a screen reader.

Adding that information to the Help information would make it possible for keyboard users to know this information.

Another useful possibility would be to add a notice that appears on :focus on the textarea that exposed those keyboard shortcuts for visual users.

Attachments (4)

27024.patch (1.5 KB) - added by joedolson 11 years ago.
Add help text for accessibility of TinyMCE toolbar.
27024.2.patch (1.5 KB) - added by joedolson 11 years ago.
Updated patch added, text modified.
27024.3.diff (1.5 KB) - added by jorbin 11 years ago.
27024.diff (1.0 KB) - added by helen 11 years ago.

Download all attachments as: .zip

Change History (13)

@joedolson
11 years ago

Add help text for accessibility of TinyMCE toolbar.

#1 @wonderboymusic
11 years ago

  • Keywords has-patch added
  • Milestone changed from Awaiting Review to 3.9

Looks like an easy one to me, will have someone take a look

#2 @azaozz
11 years ago

This should probably be:

When focus is inside the Visual editor, use Alt+F10...

@joedolson
11 years ago

Updated patch added, text modified.

@jorbin
11 years ago

#3 @jorbin
11 years ago

  • Focuses docs added
  • Keywords needs-docs added

I don't think we should call this out for Accessibility since it's not specific for use with accessible technologies. It's a keyboard shortcut. Added a patch to change the label. If there was a "needs-spell-check" tag, I would add that :)

Adding the needs-docs tag so that the docs team takes a look.

#4 @joedolson
11 years ago

That seems totally reasonable to me; labeling things as accessibility is pretty much automatic for me -- sometimes too automatic!

Should 'Shortcut' have initial lowercase, for consistency with "Post editor"?

#5 @nacin
11 years ago

  • Keywords needs-patch added; has-patch needs-docs removed

I think this should go into the TinyMCE help dialog specifically. Probably with the rest of the keyboard shortcuts.

@helen
11 years ago

#6 @helen
11 years ago

  • Keywords has-patch added; needs-patch removed

Riffed on jorbin's text and put it in the TinyMCE help dialog instead.

#7 @joedolson
11 years ago

I think that's a problem - it means that the instructions to access the menu bar can only be accessed if you can access the menu bar, since the TinyMCE help button is part of that bar.

A bit paradoxical, in practice.

#8 @nacin
11 years ago

Yeah, touché.

#9 @nacin
11 years ago

  • Owner set to nacin
  • Resolution set to fixed
  • Status changed from new to closed

In 28029:

Expose toolbar keyboard shortcut in Help documentation for TinyMCE.

props joedolson, jorbin.
fixes #27024.

Note: See TracTickets for help on using tickets.