Changeset 39414 for branches/4.7/src/wp-content/themes/twentyseventeen/inc/template-functions.php

Timestamp:

12/02/2016 05:19:44 AM (7 years ago)

Author:

joemcgill

Message:

Twenty Seventeen: Better handling of custom headers when no image is set.

This ensures that a standard header is shown on the front page whenever
a header video is set without a header image if the video doesn't load,
e.g., on mobile sizes or if the JS doesn't fire.

This adds a new class, .has-header-video that is added whenever the
wp-custom-header-video-loaded event is fired, which is then used to style
the custom headers along with .has-header-image whenever a header image
is available. This also changes the class name on the custom header media
wrapping div from .custom-header-image to .custom-header-media.

Props laurelfulford, joemcgill.
Fixes #38995 for 4.7.

Location:

branches/4.7

Files:

• . (modified) (1 prop)
• src/wp-content/themes/twentyseventeen/inc/template-functions.php (modified) (1 diff)

branches/4.7/src/wp-content/themes/twentyseventeen/inc/template-functions.php

@@ -36 +36 @@
 
     // Add a class if there is a custom header.
-    if ( has_header_image() || has_header_video() && is_front_page() ) {
+    if ( has_header_image() ) {
         $classes[] = 'has-header-image';
     }