Changeset 40458 for trunk/src/wp-content/themes/twentyseventeen/template-parts/post/content-excerpt.php

Timestamp:

04/17/2017 10:08:55 AM (7 years ago)

Author:

swissspidy

Message:

Twenty Seventeen: Correct heading hierarchy for posts on the front page.

When the posts page is on the front page or within a front page section, the heading hierarchy for the individual post titles needs to be adjusted accordingly.

Props joedolson, celloexpressions, davidakennedy.
Fixes #40264.

File:

• trunk/src/wp-content/themes/twentyseventeen/template-parts/post/content-excerpt.php (modified) (1 diff)

trunk/src/wp-content/themes/twentyseventeen/template-parts/post/content-excerpt.php

@@ -31 +31 @@
         <?php endif; ?>
 
-        <?php the_title( sprintf( '<h2 class="entry-title"><a href="%s" rel="bookmark">', esc_url( get_permalink() ) ), '</a></h2>' ); ?>
+        <?php if ( is_front_page() && ! is_home() ) {
+
+            // The excerpt is being displayed within a front page section, so it's a lower hierarchy than h2.
+            the_title( sprintf( '<h3 class="entry-title"><a href="%s" rel="bookmark">', esc_url( get_permalink() ) ), '</a></h3>' );
+        } else {
+            the_title( sprintf( '<h2 class="entry-title"><a href="%s" rel="bookmark">', esc_url( get_permalink() ) ), '</a></h2>' );
+        } ?>
     </header><!-- .entry-header -->