#29166 closed defect (bug) (fixed)
Update TinyMCE to 4.1.3
Reported by: | azaozz | Owned by: | azaozz |
---|---|---|---|
Milestone: | 4.0 | Priority: | normal |
Severity: | normal | Version: | 4.0 |
Component: | TinyMCE | Keywords: | has-patch |
Focuses: | Cc: |
Description
Was released few days ago.
Attachments (2)
Change History (10)
#4
follow-up:
↓ 6
@
10 years ago
- Keywords has-patch added
[29471] completely broke keyboard accessibility for the add media panel (see #23560), as focus is blurred when it's opened (so users can't even get into it). We need to do the try-to-hide-an-onscreen-keyboard part before setting focus on the close link, or keyboard users whose device happens to support touch basically can't use the media modal at all.
This seems like the more relevant ticket to reopen. 29166.diff fixes.
#6
in reply to:
↑ 4
;
follow-up:
↓ 8
@
10 years ago
Replying to celloexpressions:
Yep, patch works well. Btw, why are we focusing the Close button? Looks kind of bad :)
Shouldn't we focus a div that has tabindex="-1"
and some explanation in a screen-reader-text
or aria-describedby
that a modal has just opened?
#8
in reply to:
↑ 6
@
10 years ago
Replying to azaozz:
Replying to celloexpressions:
Yep, patch works well. Btw, why are we focusing the Close button? Looks kind of bad :)
Shouldn't we focus a div that has
tabindex="-1"
and some explanation in ascreen-reader-text
oraria-describedby
that a modal has just opened?
That's the best place to put focus, also being consistent with other places in wp-admin, so that the focus is visible (keyboard accessibility applying primarily to sighted users who are unable to use a mouse). #29012 made it look good, but it had to be reverted because the colorful hover/focus styling was disliked, so the plan is to come up with a new pattern for close icons and use it here too.
In 29458: