Changeset 40374 for branches/4.7

Timestamp:

04/04/2017 06:39:15 AM (8 years ago)

Author:

swissspidy

Message:

Twenty Seventeen: Use esc_attr_e() for translatable strings in HTML attributes.

Props bor0.
Fixes #40216.

Merges [40311] to the 4.7 branch.

Location:

branches/4.7

Files:

• . (modified) (1 prop)
• src/wp-content/themes/twentyseventeen/footer.php (modified) (1 diff)
• src/wp-content/themes/twentyseventeen/template-parts/navigation/navigation-top.php (modified) (1 diff)

branches/4.7/src/wp-content/themes/twentyseventeen/footer.php

@@ -23 +23 @@
 
                 if ( has_nav_menu( 'social' ) ) : ?>
-                    <nav class="social-navigation" role="navigation" aria-label="<?php _e( 'Footer Social Links Menu', 'twentyseventeen' ); ?>">
+                    <nav class="social-navigation" role="navigation" aria-label="<?php esc_attr_e( 'Footer Social Links Menu', 'twentyseventeen' ); ?>">
                         <?php
                             wp_nav_menu( array(

branches/4.7/src/wp-content/themes/twentyseventeen/template-parts/navigation/navigation-top.php

@@ -10 +10 @@
 
 ?>
-<nav id="site-navigation" class="main-navigation" role="navigation" aria-label="<?php _e( 'Top Menu', 'twentyseventeen' ); ?>">
+<nav id="site-navigation" class="main-navigation" role="navigation" aria-label="<?php esc_attr_e( 'Top Menu', 'twentyseventeen' ); ?>">
     <button class="menu-toggle" aria-controls="top-menu" aria-expanded="false"><?php echo twentyseventeen_get_svg( array( 'icon' => 'bars' ) ); echo twentyseventeen_get_svg( array( 'icon' => 'close' ) ); _e( 'Menu', 'twentyseventeen' ); ?></button>
     <?php wp_nav_menu( array(