[wp-trac] [WordPress Trac] #29166: Update TinyMCE to 4.1.3
WordPress Trac
noreply at wordpress.org
Tue Aug 12 22:12:27 UTC 2014
#29166: Update TinyMCE to 4.1.3
--------------------------+---------------------
Reporter: azaozz | Owner: azaozz
Type: defect (bug) | Status: closed
Priority: normal | Milestone: 4.0
Component: TinyMCE | Version: trunk
Severity: normal | Resolution: fixed
Keywords: has-patch | Focuses:
--------------------------+---------------------
Comment (by celloexpressions):
Replying to [comment:6 azaozz]:
> Replying to [comment:4 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?
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.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/29166#comment:8>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list