Ticket #31527: 31527.twentyfourteen.searchtoggle.2.patch

wp-content/themes/twentyfourteen/header.php

@@ -45 +45 @@
                         <h1 class="site-title"><a href="<?php echo esc_url( home_url( '/' ) ); ?>" rel="home"><?php bloginfo( 'name' ); ?></a></h1>
 
                         <div class="search-toggle">
-                                <a href="#search-container" class="screen-reader-text"><?php _e( 'Search', 'twentyfourteen' ); ?></a>
+                                <a href="#search-container" class="screen-reader-text" aria-expanded="false" aria-controls="search-container"><?php _e( 'Search', 'twentyfourteen' ); ?></a>
                         </div>
 
                         <nav id="primary-navigation" class="site-navigation primary-navigation" role="navigation">

wp-content/themes/twentyfourteen/js/functions.js

@@ -69 +69 @@
                 // Search toggle.
                 $( '.search-toggle' ).on( 'click.twentyfourteen', function( event ) {
                         var that    = $( this ),
-                                wrapper = $( '.search-box-wrapper' );
+                                wrapper = $( '#search-container' ),
+                                container = that.find( 'a' );
 
                         that.toggleClass( 'active' );
                         wrapper.toggleClass( 'hide' );
 
+                        if ( that.hasClass( 'active' ) ) {
+                                container.attr( 'aria-expanded', 'true' );
+                        } else {
+                                container.attr( 'aria-expanded', 'false' );
+                        }
+
                         if ( that.is( '.active' ) || $( '.search-toggle .screen-reader-text' )[0] === event.target ) {
                                 wrapper.find( '.search-field' ).focus();
                         }