[wp-trac] [WordPress Trac] #36267: Thickbox close button accessibility

[WordPress Trac]

#36267: Thickbox close button accessibility
---------------------------------------+--------------------------------
 Reporter:  afercia                    |       Owner:  afercia
     Type:  defect (bug)               |      Status:  assigned
 Priority:  normal                     |   Milestone:  4.6
Component:  Administration             |     Version:
 Severity:  normal                     |  Resolution:
 Keywords:  has-patch has-screenshots  |     Focuses:  ui, accessibility
---------------------------------------+--------------------------------
Changes (by afercia):

 * keywords:  has-patch => has-patch has-screenshots


Comment:

 Patch still applies, tested with a couple of plugins that use Thickbox and
 seems everything is OK. I'd propose to add just a small CSS change. In the
 screenshot below, a Thickbox example before the patch: notice the focus
 style around the "Close" link, it's the standard focus style for links

 [[Image(https://cldup.com/DqYC2ZgRaX.png)]]

 After the patch, the link will be a button and will lose the focus style
 that should be added again in `thickbox.css`,

--
Ticket URL: <https://core.trac.wordpress.org/ticket/36267#comment:5>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform