Twenty Twenty: Remove aria-expanded on search modal close button.
The Twenty Twenty search modal uses aria-expanded correctly on the Search button, but toggles the aria-expanded value on the 'close' button. The close button is a different control, however, so the attribute value toggling is uneven. The close button does not toggle any behavior; it only closes a control that has already been opened. Props alexstine, audrasjb Fixes #52355 Built from https://develop.svn.wordpress.org/trunk@50033 git-svn-id: http://core.svn.wordpress.org/trunk@49734 1a063a9b-81f0-0310-95a4-ce76da25c4cd
This commit is contained in:
parent
ee751b9779
commit
830d3942a0
|
@ -676,6 +676,9 @@ twentytwentyDomReady( function() {
|
||||||
/* Toggle an attribute ----------------------- */
|
/* Toggle an attribute ----------------------- */
|
||||||
|
|
||||||
function twentytwentyToggleAttribute( element, attribute, trueVal, falseVal ) {
|
function twentytwentyToggleAttribute( element, attribute, trueVal, falseVal ) {
|
||||||
|
if ( element.classList.contains( 'close-search-toggle' ) ) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
if ( trueVal === undefined ) {
|
if ( trueVal === undefined ) {
|
||||||
trueVal = true;
|
trueVal = true;
|
||||||
}
|
}
|
||||||
|
|
|
@ -22,7 +22,7 @@
|
||||||
);
|
);
|
||||||
?>
|
?>
|
||||||
|
|
||||||
<button class="toggle search-untoggle close-search-toggle fill-children-current-color" data-toggle-target=".search-modal" data-toggle-body-class="showing-search-modal" data-set-focus=".search-modal .search-field" aria-expanded="false">
|
<button class="toggle search-untoggle close-search-toggle fill-children-current-color" data-toggle-target=".search-modal" data-toggle-body-class="showing-search-modal" data-set-focus=".search-modal .search-field">
|
||||||
<span class="screen-reader-text"><?php _e( 'Close search', 'twentytwenty' ); ?></span>
|
<span class="screen-reader-text"><?php _e( 'Close search', 'twentytwenty' ); ?></span>
|
||||||
<?php twentytwenty_the_theme_svg( 'cross' ); ?>
|
<?php twentytwenty_the_theme_svg( 'cross' ); ?>
|
||||||
</button><!-- .search-toggle -->
|
</button><!-- .search-toggle -->
|
||||||
|
|
|
@ -13,7 +13,7 @@
|
||||||
*
|
*
|
||||||
* @global string $wp_version
|
* @global string $wp_version
|
||||||
*/
|
*/
|
||||||
$wp_version = '5.7-alpha-50032';
|
$wp_version = '5.7-alpha-50033';
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Holds the WordPress DB revision, increments when changes are made to the WordPress DB schema.
|
* Holds the WordPress DB revision, increments when changes are made to the WordPress DB schema.
|
||||||
|
|
Loading…
Reference in New Issue