Changeset 30393 for trunk/src/wp-admin/menu-header.php

Timestamp:

11/19/2014 07:36:43 PM (10 years ago)

Author:

helen

Message:

Accessibility: hide admin menu separators from screen readers.

props afercia. fixes #30010.

File:

• trunk/src/wp-admin/menu-header.php (modified) (3 diffs)

trunk/src/wp-admin/menu-header.php

@@ -53 +53 @@
         $class = array();
         $aria_attributes = '';
+        $aria_hidden = '';
+        $is_separator = false;
 
         if ( $first ) {
@@ -80 +82 @@
         $img = $img_style = '';
         $img_class = ' dashicons-before';
+
+        if ( false !== strpos( $class, 'wp-menu-separator' ) ) {
+            $is_separator = true;
+        }
 
         /*
@@ -105 +111 @@
         $title = wptexturize( $item[0] );
 
-        echo "\n\t<li$class$id>";
-
-        if ( false !== strpos( $class, 'wp-menu-separator' ) ) {
+        // hide separators from screen readers
+        if ( $is_separator ) {
+            $aria_hidden = ' aria-hidden="true"';
+        }
+
+        echo "\n\t<li$class$id$aria_hidden>";
+
+        if ( $is_separator ) {
             echo '<div class="separator"></div>';
         } elseif ( $submenu_as_parent && ! empty( $submenu_items ) ) {