Changeset 36915 for trunk/src/wp-includes/class-wp-customize-manager.php

Timestamp:

03/09/2016 11:43:49 PM (10 years ago)

Author:

obenland

Message:

Customize: Only add custom logo’s header text control if needed.

Dissolves WP_CustomLogo and adopts a structure similar to custom header and background (See _delete_attachment_theme_mod()).
The option to hide header text only gets added if it’s not already part of custom header, and only if selectors have been registered when theme support for custom logos was declared. Themes can add postMessage support for it as well.

Example:

add_theme_support( 'custom-logo', array(
    'size' => ‘large’,
    'header-text' => array( 'site-title', 'site-description' ),
) );

See #33755.

File:

• trunk/src/wp-includes/class-wp-customize-manager.php (modified) (1 diff)

trunk/src/wp-includes/class-wp-customize-manager.php

@@ -1923 +1923 @@
         ) );
 
-        // Add a setting to hide header text if the theme isn't supporting the feature itself.
-        // @todo
-        if ( ! current_theme_supports( 'custom-header' ) ) {
+        // Add a setting to hide header text if the theme doesn't support custom headers.
+        if ( ! current_theme_supports( 'custom-header', 'header-text' ) ) {
             $this->add_setting( 'header_text', array(
+                'theme_supports'    => array( 'custom-logo', 'header-text' ),
                 'default'           => 1,
                 'sanitize_callback' => 'absint',
-                'transport'         => 'postMessage',
             ) );