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
This commit is contained in:
parent
7ba0908947
commit
c8b86fdd52
|
@ -21,9 +21,6 @@
|
||||||
container.find( '.current-menu-ancestor > button' ).addClass( 'toggled-on' );
|
container.find( '.current-menu-ancestor > button' ).addClass( 'toggled-on' );
|
||||||
container.find( '.current-menu-ancestor > .sub-menu' ).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 ) {
|
container.find( '.dropdown-toggle' ).click( function( e ) {
|
||||||
var _this = $( this ),
|
var _this = $( this ),
|
||||||
screenReaderSpan = _this.find( '.screen-reader-text' );
|
screenReaderSpan = _this.find( '.screen-reader-text' );
|
||||||
|
|
Loading…
Reference in New Issue