Changeset 53051 for trunk/src/wp-content/themes/twentytwenty/template-parts/modal-menu.php

Timestamp:

04/01/2022 05:54:30 PM (3 years ago)

Author:

joedolson

Message:

Twenty Twenty: Fix aria-expanded handling in search toggle.

Transform search into a dialog role and fix the handling of aria-expanded to synchronize mobile and desktop buttons.

Props utz119, alexstine, mukesh27, hareesh-pillai, sabernhardt, audrasjb, afercia.
Fixes #53951.

File:

• trunk/src/wp-content/themes/twentytwenty/template-parts/modal-menu.php (modified) (1 diff)

trunk/src/wp-content/themes/twentytwenty/template-parts/modal-menu.php

@@ -18 +18 @@
             <div class="menu-top">
 
-                <button class="toggle close-nav-toggle fill-children-current-color" data-toggle-target=".menu-modal" data-toggle-body-class="showing-menu-modal" aria-expanded="false" data-set-focus=".menu-modal">
+                <button class="toggle close-nav-toggle fill-children-current-color" data-toggle-target=".menu-modal" data-toggle-body-class="showing-menu-modal" data-set-focus=".menu-modal">
                     <span class="toggle-text"><?php _e( 'Close Menu', 'twentytwenty' ); ?></span>
                     <?php twentytwenty_the_theme_svg( 'cross' ); ?>