WordPress.org

Make WordPress Core

Ticket #38737: 38737.2.diff

File 38737.2.diff, 5.6 KB (added by bradyvercher, 5 years ago)
  • src/wp-content/themes/twentyfourteen/header.php

     
    3232
    3333<body <?php body_class(); ?>>
    3434<div id="page" class="hfeed site">
    35         <?php if ( function_exists( 'has_header_video' ) && has_header_video() ) : ?>
     35        <?php if ( is_front_page() && function_exists( 'has_header_video' ) && has_header_video() ) : ?>
    3636                <div id="site-header">
    3737                        <?php the_custom_header_markup(); ?>
    3838                </div>
  • src/wp-content/themes/twentyfourteen/inc/customizer.php

     
    1919        $wp_customize->get_setting( 'blogname' )->transport          = 'postMessage';
    2020        $wp_customize->get_setting( 'blogdescription' )->transport   = 'postMessage';
    2121        $wp_customize->get_setting( 'header_textcolor' )->transport  = 'postMessage';
    22         $wp_customize->get_setting( 'header_image' )->transport      = 'postMessage';
    23         $wp_customize->get_setting( 'header_image_data' )->transport = 'postMessage';
    2422
    2523        if ( isset( $wp_customize->selective_refresh ) ) {
    2624                $wp_customize->selective_refresh->add_partial( 'blogname', array(
  • src/wp-content/themes/twentyseventeen/inc/customizer.php

     
    1616        $wp_customize->get_setting( 'blogname' )->transport          = 'postMessage';
    1717        $wp_customize->get_setting( 'blogdescription' )->transport   = 'postMessage';
    1818        $wp_customize->get_setting( 'header_textcolor' )->transport  = 'postMessage';
    19         $wp_customize->get_setting( 'header_image' )->transport      = 'postMessage';
    20         $wp_customize->get_setting( 'header_image_data' )->transport = 'postMessage';
    2119
    2220        $wp_customize->selective_refresh->add_partial( 'blogname', array(
    2321                'selector' => '.site-title a',
  • src/wp-content/themes/twentyseventeen/template-parts/header/header-image.php

     
    1010
    1111?>
    1212<div class="custom-header">
    13         <?php
    14         $header_image = get_header_image();
    1513
    16         // Check if Custom Header image has been added.
    17         if ( has_custom_header() ) :
    18         ?>
     14        <div class="custom-header-image">
     15                <?php the_custom_header_markup(); ?>
     16        </div>
    1917
    20                 <?php // Output the full custom header - video and/or image fallback. ?>
    21                 <div class="custom-header-image">
    22                         <?php the_custom_header_markup(); ?>
    23                 </div>
    24                 <?php get_template_part( 'template-parts/header/site', 'branding' ); ?>
     18        <?php get_template_part( 'template-parts/header/site', 'branding' ); ?>
    2519
    26         <?php else : ?>
    27 
    28                 <?php // Otherwise, show a blank header. ?>
    29                 <div class="custom-header-simple">
    30                         <?php get_template_part( 'template-parts/header/site', 'branding' ); ?>
    31                 </div><!-- .custom-header-simple -->
    32 
    33         <?php endif; ?>
    34 
    3520</div><!-- .custom-header -->
  • src/wp-includes/class-wp-customize-manager.php

     
    34623462                        'theme_supports' => 'custom-header',
    34633463                ) ) );
    34643464
     3465                // Switch image settings to post message when video support is enabled.
     3466                if ( current_theme_supports( 'custom-header', 'video' ) ) {
     3467                        $this->get_setting( 'header_image' )->transport = 'postMessage';
     3468                        $this->get_setting( 'header_image_data' )->transport = 'postMessage';
     3469                }
     3470
    34653471                $this->add_control( new WP_Customize_Media_Control( $this, 'header_video', array(
    34663472                        'theme_supports' => array( 'custom-header', 'video' ),
    34673473                        'label'          => __( 'Header Video' ),
  • src/wp-includes/theme.php

     
    14101410/**
    14111411 * Retrieve the markup for a custom header.
    14121412 *
     1413 * The container div will always be returned in the Customizer preview.
     1414 *
    14131415 * @since 4.7.0
    14141416 *
    1415  * @return string|false The markup for a custom header on success. False if not.
     1417 * @return string The markup for a custom header on success.
    14161418 */
    14171419function get_custom_header_markup() {
    1418         if ( ! has_custom_header() ) {
    1419                 return false;
     1420        if ( ! has_custom_header() && ! is_customize_preview() ) {
     1421                return '';
    14201422        }
    14211423
    14221424        return sprintf(
     
    14281430/**
    14291431 * Print the markup for a custom header.
    14301432 *
     1433 * A container div will always be printed in the Customizer preview.
     1434 *
    14311435 * @since 4.7.0
    14321436 */
    1433 function the_custom_header_markup() {
    1434         if ( ! $custom_header = get_custom_header_markup() ) {
     1437function the_custom_header_markup( $args = array() ) {
     1438        $custom_header = get_custom_header_markup();
     1439        if ( empty( $custom_header ) ) {
    14351440                return;
    14361441        }
     1442
    14371443        echo $custom_header;
    14381444
    1439         if ( has_header_video() && is_front_page() ) {
     1445        if ( is_front_page() && ( has_header_video() || is_customize_preview() ) ) {
    14401446                wp_enqueue_script( 'wp-custom-header' );
    14411447                wp_localize_script( 'wp-custom-header', '_wpCustomHeaderSettings', get_header_video_settings() );
    14421448        }