WordPress.org

Make WordPress Core

Changeset 39232


Ignore:
Timestamp:
11/14/2016 10:33:40 PM (5 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:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-content/themes/twentyseventeen/assets/js/navigation.js

    r39221 r39232  
    2121        container.find( '.current-menu-ancestor > button' ).addClass( 'toggled-on' );
    2222        container.find( '.current-menu-ancestor > .sub-menu' ).addClass( 'toggled-on' );
    23 
    24         // Add menu items with submenus to aria-haspopup="true".
    25         container.find( '.menu-item-has-children, .page_item_has_children' ).attr( 'aria-haspopup', 'true' );
    2623
    2724        container.find( '.dropdown-toggle' ).click( function( e ) {
Note: See TracChangeset for help on using the changeset viewer.