Changeset 53051 for trunk/src/wp-content/themes/twentytwenty/template-parts/modal-search.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-search.php (modified) (1 diff)

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

@@ -9 +9 @@
 
 ?>
-<div class="search-modal cover-modal header-footer-group" data-modal-target-string=".search-modal">
+<div class="search-modal cover-modal header-footer-group" data-modal-target-string=".search-modal" role="dialog" aria-modal="true" aria-label="<?php esc_attr_e( 'Search', 'twentytwenty' ); ?>">
 
     <div class="search-modal-inner modal-inner">