Changeset 39232

Timestamp:

11/14/2016 10:33:40 PM (4 years ago)

Author:

karmatosed

Message:

Twenty Seventeen: Fix for mobile menu parent items requiring double tap

After further review from the accessibility team, it was determined that the menu items with sub-menus don't need the aria-haspopup attribute.

Props davidakennedy, celloexpressions, voldemortensen, afercia
See #38397

File:

• trunk/src/wp-content/themes/twentyseventeen/assets/js/navigation.js (modified) (1 diff)

trunk/src/wp-content/themes/twentyseventeen/assets/js/navigation.js

@@ -21 +21 @@
         container.find( '.current-menu-ancestor > button' ).addClass( 'toggled-on' );
         container.find( '.current-menu-ancestor > .sub-menu' ).addClass( 'toggled-on' );
-
-        // Add menu items with submenus to aria-haspopup="true".
-        container.find( '.menu-item-has-children, .page_item_has_children' ).attr( 'aria-haspopup', 'true' );
 
         container.find( '.dropdown-toggle' ).click( function( e ) {