Changeset 50933 for trunk/src/wp-content/themes/twentytwenty/searchform.php

Timestamp:

05/19/2021 06:38:53 PM (3 years ago)

Author:

davidbaumwald

Message:

Twenty Twenty: Correct label attribute references to aria_label in get_search_form.

In the bundled Twenty Twenty theme, a label argument was passed to get_search_form but used as the aria_label on the form. This change updates the argument name to aria_label and maintains backwards compatibility for usage of the original label argument.

Props poena, sabernhardt, ipulc2, kishanjasani, mukesh27.
Fixes #51877.

File:

• trunk/src/wp-content/themes/twentytwenty/searchform.php (modified) (1 diff)

trunk/src/wp-content/themes/twentytwenty/searchform.php

@@ -18 +18 @@
 $twentytwenty_unique_id = twentytwenty_unique_id( 'search-form-' );
 
-$twentytwenty_aria_label = ! empty( $args['label'] ) ? 'aria-label="' . esc_attr( $args['label'] ) . '"' : '';
+$twentytwenty_aria_label = ! empty( $args['aria_label'] ) ? 'aria-label="' . esc_attr( $args['aria_label'] ) . '"' : '';
+// Backward compatibility, in case a child theme template uses a `label` argument.
+if ( empty( $twentytwenty_aria_label ) && ! empty( $args['label'] ) ) {
+    $twentytwenty_aria_label = 'aria-label="' . esc_attr( $args['label'] ) . '"';
+}
 ?>
 <form role="search" <?php echo $twentytwenty_aria_label; // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped -- Escaped above. ?> method="get" class="search-form" action="<?php echo esc_url( home_url( '/' ) ); ?>">