WordPress.org

Make WordPress Core

Opened 3 years ago

Closed 3 years ago

#20834 closed defect (bug) (fixed)

Tabbing issues with TinyMCE

Reported by: jkudish Owned by: azaozz
Milestone: 3.5 Priority: normal
Severity: normal Version: 3.4
Component: Accessibility Keywords: needs-patch
Focuses: Cc:

Description (last modified by azaozz)

I am seeing different tabbing behaviours across different browsers with the visual editor/TinyMCE:

  • Chrome 19: You can only tab into the editor from the title field when in visual mode once. After you focus out of TinyMCE and back into the title field, you cannot tab back into TinyMCE
  • Firefox 12: You can't tab into the editor at all, tabbing from the title field, brings you to the publish box
  • Safari 5.1.7: Tabbing in/out of the editor always works properly

This is especially problematic for blind users/accessibility. Would be great to standardize that behaviour across all browsers :)

Attachments (1)

20834.patch (1.3 KB) - added by azaozz 3 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 @azaozz3 years ago

  • Milestone changed from Awaiting Review to Future Release

comment:2 @azaozz3 years ago

  • Component changed from TinyMCE to Accessibility
  • Description modified (diff)

comment:3 follow-up: @grahamarmfield3 years ago

This is a serious issue for blind users yes, but also for sighted users who rely on keyboard interaction.

The tab order of the edit page/edit post screen is also very confusing in my view - it would be good to have a review and get a sensible tab order in place across all browsers.

comment:4 in reply to: ↑ 3 @azaozz3 years ago

Replying to grahamarmfield:

...The tab order of the edit page/edit post screen is also very confusing in my view...

Yes, been meaning to change that for a while. As far as I can see most admin pages can have a "natural" tab order, i.e. the HTML elements flow the same way in the source and on the screen and don't need any tabindex. The only exception is the top toolbar that is outputted at the bottom but shown at the top.

Thinking to remove all tabindex everywhere and then add it only where necessary (after testing). Would you be willing to test this?

Version 0, edited 3 years ago by azaozz (next)

comment:5 @azaozz3 years ago

  • Milestone changed from Future Release to 3.5

comment:6 @grahamarmfield3 years ago

Yes I could help with that - how would I access an area to test it in?

comment:7 follow-up: @grahamarmfield3 years ago

Can we look at improving the visibility of focus around the admin screens? Or should I raise a separate ticket for that?

comment:8 in reply to: ↑ 7 @azaozz3 years ago

Replying to grahamarmfield:

Of course. Think a separate ticket for that would be better so we can track all changes and bugs specifically.

Last edited 3 years ago by azaozz (previous) (diff)

comment:9 @azaozz3 years ago

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

In [21300]:

TinyMCE: fix tabbig in and out of the editor, fixes #20834

comment:10 @azaozz3 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

This also needs to support tabbing from Title directly to the textarea when the Text editor is active.

@azaozz3 years ago

comment:11 @azaozz3 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

In [21303]:

Add tabbing from Title directly to the textarea when the Text editor is active, fixes #20834

Note: See TracTickets for help on using tickets.