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

[WordPress Trac]

#36267: Thickbox close button accessibility
-------------------------------+-----------------------------
 Reporter:  afercia            |      Owner:
     Type:  defect (bug)       |     Status:  new
 Priority:  normal             |  Milestone:  Awaiting Review
Component:  Administration     |    Version:
 Severity:  normal             |   Keywords:
  Focuses:  ui, accessibility  |
-------------------------------+-----------------------------
 Recently, [36964] changed the Thickbox button in a real button element but
 there's one more case where it's still a non-link link `a href='#'` that
 should be updated as well. Was not handled in #33305 because, as far as I
 can tell, core is not using "inline" content for Thickbox and was out of
 the scope of that ticket but plugins are using it and they get just an
 empty link with no content at all.
 Screen readers will read it out a "link".
 I'd propose to use the same markup already used for the other cases,
 making it a `<button>` element with some `screen-reader-text` inside.

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