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

Timestamp:

11/14/2016 06:40:08 PM (10 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 -->