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

[WordPress Trac]

#30468: wplink modal accessibility
-------------------------------------+-------------------------------------
 Reporter:  afercia                  |       Owner:
     Type:  defect (bug)             |      Status:  new
 Priority:  high                     |   Milestone:  4.2
Component:  Editor                   |     Version:  4.0
 Severity:  normal                   |  Resolution:
 Keywords:  needs-testing dev-       |     Focuses:  ui, accessibility,
  feedback                           |  javascript
-------------------------------------+-------------------------------------
Changes (by afercia):

 * keywords:  needs-testing dev-feedback needs-refresh punt => needs-testing
     dev-feedback


Comment:

 Updated patch for the new iteration of the wpLink modal.
 - re-introduce all the a11y features from previous patch
 - some non strictly related JS refactoring (braces) sorry for that, had to
 run and quickly merged the previous patch :)
 - CSS adjustments for the spinner position
 - please check the modal height on small screens, might need some tweaking

 For accessibility reasons, the first input field should have initial
 focus. Please consider to never skip a form field thus forcing users to
 reverse tabbing to "discover" there's something else there.
 Will work correctly when empty of course and when there's some selected
 text the it will be announced as "selected {text}".

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