From c8b86fdd5215dc65c757c89874d37d38d00c379c Mon Sep 17 00:00:00 2001 From: Tammie Lister Date: Mon, 14 Nov 2016 22:33:40 +0000 Subject: [PATCH] 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 git-svn-id: https://develop.svn.wordpress.org/trunk@39232 602fd350-edb4-49c9-b593-d223f7449a82 --- src/wp-content/themes/twentyseventeen/assets/js/navigation.js | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/wp-content/themes/twentyseventeen/assets/js/navigation.js b/src/wp-content/themes/twentyseventeen/assets/js/navigation.js index b20e0c92d3..c1af2af3fa 100644 --- a/src/wp-content/themes/twentyseventeen/assets/js/navigation.js +++ b/src/wp-content/themes/twentyseventeen/assets/js/navigation.js @@ -21,9 +21,6 @@ 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 ) { var _this = $( this ), screenReaderSpan = _this.find( '.screen-reader-text' );