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

WordPress Trac noreply at wordpress.org
Mon Jun 1 21:12:48 UTC 2015


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

Comment (by joedolson):

 So, I have two methods for solving this problem. The first is simple; add
 an exception to `class-wp-admin-bar.php` so that the node ID 'search' uses
 a div. (32495.3.patch)

 The second is to modify the admin bar node object to have a 'wrapper'
 parameter with default value 'a', that is set to 'div' for the search
 object in `admin-bar.php`. (32495.4.patch)

 The latter is more elegant to me, and shouldn't break anything in core;
 but obviously needs some testing.

 If there are plug-ins or themes adding content to the adminbar that
 depends on a <div> wrapper, the latter solution allows them to keep this
 while still providing accessibility support for more common circumstances.

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


More information about the wp-trac mailing list