Changeset 30072 for trunk/src/wp-content/themes/twentyfifteen/header.php

Timestamp:

10/28/2014 09:02:02 PM (5 years ago)

Author:

iandstewart

Message:

Twenty Fifteen: Use a heading heirarchy to provide a better navigation experience when using screenreading software.

Props davidakennedy, rianrietveld, iamtakashi, afercia, davidakennedy, fixes #30065.

File:

• trunk/src/wp-content/themes/twentyfifteen/header.php (modified) (1 diff)

trunk/src/wp-content/themes/twentyfifteen/header.php

@@ -30 +30 @@
         <header id="masthead" class="site-header" role="banner">
             <div class="site-branding">
-                <h1 class="site-title"><a href="<?php echo esc_url( home_url( '/' ) ); ?>" rel="home"><?php bloginfo( 'name' ); ?></a></h1>
-                <p class="site-description"><?php bloginfo( 'description' ); ?></p>
+                <?php
+                    if ( is_front_page() && is_home() ) : ?>
+                        <h1 class="site-title"><a href="<?php echo esc_url( home_url( '/' ) ); ?>" rel="home"><?php bloginfo( 'name' ); ?></a></h1>
+                    <?php else : ?>
+                        <p class="site-title"><a href="<?php echo esc_url( home_url( '/' ) ); ?>" rel="home"><?php bloginfo( 'name' ); ?></a></p>
+                    <?php endif;
+
+                    $description = get_bloginfo( 'description', 'display' );
+                    if ( ! empty( $description ) ) : ?>
+                        <p class="site-description"><?php echo esc_html( $description ); ?></p>
+                    <?php endif;
+                ?>
                 <button class="secondary-toggle"><?php esc_html_e( 'Menu and widgets', 'twentyfifteen' ); ?></button>
             </div><!-- .site-branding -->