[wp-trac] [WordPress Trac] #32495: WP Admin bar multi-level menu accessibility

[WordPress Trac]

#32495: WP Admin bar multi-level menu accessibility
--------------------------+--------------------------------------------
 Reporter:  joedolson     |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  high          |   Milestone:  4.6
Component:  General       |     Version:
 Severity:  normal        |  Resolution:
 Keywords:  has-patch     |     Focuses:  accessibility, administration
--------------------------+--------------------------------------------

Comment (by afercia):

 As @ocean90 pointed out [attachment:32495.2.diff] would change the current
 behaviour if someone is defining a tabindex with an empty string as value.
 So [attachment:32495.3.diff] takes a different approach that should
 preserve the current behavior ''and'' allow a tabindex with a value of
 `0`, which is the value we need for accessibility.

 Wondering if this could be simplified and made more readable, not sure
 there's really the need to keep the two ternary expressions. Any thoughts
 welcome.

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