Changeset 39227 for trunk/src/wp-content/themes/twentyseventeen/template-parts/header/header-image.php

Timestamp:

11/14/2016 06:40:08 PM (9 years ago)

Author:

westonruter

Message:

Customize: Use selective refresh for custom header changes when possible.

• Use postMessage transport for header image settings by default when video headers are supported, and thus the_custom_header_markup() will necessarily be used (and thus a selective refresh partial will be available).
• Ensure that the_custom_header_markup() always outputs a container element in the customizer preview even if the header is empty.
• Ensure that edit shortcut appearing for custom header does not get positioned off-screen.

Props bradyvercher, westonruter.
See #38639.
Fixes #38737.

File:

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

trunk/src/wp-content/themes/twentyseventeen/template-parts/header/header-image.php

@@ -11 +11 @@
 ?>
 <div class="custom-header">
-    <?php
 
-    // Check if Custom Header image has been added.
-    if ( has_custom_header() ) :
-    ?>
+    <div class="custom-header-image">
+        <?php the_custom_header_markup(); ?>
+    </div>
 
-        <?php // Output the full custom header - video and/or image fallback. ?>
-        <div class="custom-header-image">
-            <?php the_custom_header_markup(); ?>
-        </div>
-        <?php get_template_part( 'template-parts/header/site', 'branding' ); ?>
-
-    <?php else : ?>
-
-        <?php // Otherwise, show a blank header. ?>
-        <div class="custom-header-simple">
-            <?php get_template_part( 'template-parts/header/site', 'branding' ); ?>
-        </div><!-- .custom-header-simple -->
-
-    <?php endif; ?>
+    <?php get_template_part( 'template-parts/header/site', 'branding' ); ?>
 
 </div><!-- .custom-header -->