forked from mirror/_s
Merge pull request #1303 from jaredcaraway/fix-aria-expanded
Remove unnecessary aria-expanded attribute on ul
This commit is contained in:
commit
55fafb9558
|
@ -25,7 +25,6 @@
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
menu.setAttribute( 'aria-expanded', 'false' );
|
|
||||||
if ( -1 === menu.className.indexOf( 'nav-menu' ) ) {
|
if ( -1 === menu.className.indexOf( 'nav-menu' ) ) {
|
||||||
menu.className += ' nav-menu';
|
menu.className += ' nav-menu';
|
||||||
}
|
}
|
||||||
|
@ -34,11 +33,9 @@
|
||||||
if ( -1 !== container.className.indexOf( 'toggled' ) ) {
|
if ( -1 !== container.className.indexOf( 'toggled' ) ) {
|
||||||
container.className = container.className.replace( ' toggled', '' );
|
container.className = container.className.replace( ' toggled', '' );
|
||||||
button.setAttribute( 'aria-expanded', 'false' );
|
button.setAttribute( 'aria-expanded', 'false' );
|
||||||
menu.setAttribute( 'aria-expanded', 'false' );
|
|
||||||
} else {
|
} else {
|
||||||
container.className += ' toggled';
|
container.className += ' toggled';
|
||||||
button.setAttribute( 'aria-expanded', 'true' );
|
button.setAttribute( 'aria-expanded', 'true' );
|
||||||
menu.setAttribute( 'aria-expanded', 'true' );
|
|
||||||
}
|
}
|
||||||
};
|
};
|
||||||
|
|
||||||
|
@ -49,7 +46,6 @@
|
||||||
if ( ! isClickInside ) {
|
if ( ! isClickInside ) {
|
||||||
container.className = container.className.replace( ' toggled', '' );
|
container.className = container.className.replace( ' toggled', '' );
|
||||||
button.setAttribute( 'aria-expanded', 'false' );
|
button.setAttribute( 'aria-expanded', 'false' );
|
||||||
menu.setAttribute( 'aria-expanded', 'false' );
|
|
||||||
}
|
}
|
||||||
} );
|
} );
|
||||||
|
|
||||||
|
|
Reference in New Issue