[wp-trac] [WordPress Trac] #51580: Application Passwords: Accessibility Improvements

[WordPress Trac]

#51580: Application Passwords: Accessibility Improvements
------------------------------------+--------------------------------------
 Reporter:  georgestephanis         |       Owner:  georgestephanis
     Type:  defect (bug)            |      Status:  reopened
 Priority:  normal                  |   Milestone:  5.6
Component:  Login and Registration  |     Version:  trunk
 Severity:  normal                  |  Resolution:
 Keywords:  has-patch               |     Focuses:  accessibility, rest-api
------------------------------------+--------------------------------------

Comment (by TimothyBlynJacobs):

 > The tabindex attribute value should be -1 though a not 0.

 Will update.

 >  In either cases, focus needs to be moved to a proper place, not sure
 what the best place is though.

 I'm not sure either. Looking at the comments table, it appears that the
 current row is instead hidden with `display: none` and the focus stays on
 the "Trash" link.

 > The readonly input field doesn't have an associated <label> element. As
 pet the accessibility coding standards, all input fields need to have a
 label associated with for/id attributes

 When I looked thru core for usages of `readonly="readonly"` the only
 instance I saw with a `<label>` was the media manager.

 > See the sample code provided by @sabernhardt above.

 Will implement.

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