[wp-trac] [WordPress Trac] #30468: TinyMCE wplink modal accessibility

[WordPress Trac]

#30468: TinyMCE wplink modal accessibility
-------------------------------------------+-----------------------------
 Reporter:  afercia                        |      Owner:
     Type:  defect (bug)                   |     Status:  new
 Priority:  normal                         |  Milestone:  Awaiting Review
Component:  TinyMCE                        |    Version:  4.0
 Severity:  normal                         |   Keywords:
  Focuses:  ui, accessibility, javascript  |
-------------------------------------------+-----------------------------
 After #28897, mainly focused on keyboard accessibility, I'd like to
 propose a second accessibility round for the wplink modal, this time more
 focused on screen readers. In the proposed patch:

 - added aria live region, aria roles, aria attributes
 - use role "application" on the link listings to avoid key strokes
 conflicts with screen readers, focus is handled
 - better focus management
 - avoid to move focus in one potentially confusing case
 - changed two non-links in buttons
 - use CSS class 'modal-open' on the body
 - CSS refinements
 - wplink.js braces

 Tested with Firefox + NVDA and (quickly) with Chrome + ChromeVox.

 Seems almost everything gets read out nicely. Accessibility Team and
 everyone interested, please test everything :) Any thoughts more than
 welcome.

 Please remember to test when "link to existing content" is open and:
 - showing recent items (default view)
 - doing a search with results
 - doing a search with no results

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