Changeset 39448

Timestamp:

12/02/2016 11:02:15 PM (10 years ago)

Author:

helen

Message:

Twenty Seventeen: Ensure header text color updates in Customizer preview when cleared

Checks if the text color has been cleared and removes the style block if it has been. Uses color styles in theme stylesheet.

Props sstoqnov, davidakennedy.

Fixes #38993 for the 4.7 branch.

Location:

branches/4.7

Files:

• . (modified) (1 prop)
• src/wp-content/themes/twentyseventeen/assets/js/customize-preview.js (modified) (3 diffs)
• src/wp-content/themes/twentyseventeen/inc/custom-header.php (modified) (1 diff)

branches/4.7/src/wp-content/themes/twentyseventeen/assets/js/customize-preview.js

@@ -9 +9 @@
         // Collect information from customize-controls.js about which panels are opening.
         wp.customize.bind( 'preview-ready', function() {
-                
+
                 // Initially hide the theme option placeholders on load
                 $( '.panel-placeholder' ).hide();
-                
+
                 wp.customize.preview.bind( 'section-highlight', function( data ) {
 
@@ -59 +59 @@
                                         position: 'absolute'
                                 });
+                                // Add class for different logo styles if title and description are hidden.
                                 $( 'body' ).addClass( 'title-tagline-hidden' );
                         } else {
+
+                                // Check if the text color has been removed and use default colors in theme stylesheet.
+                                if ( ! to.length ) {
+                                        $( '#twentyseventeen-custom-header-styles' ).remove();
+                                }
                                 $( '.site-title, .site-description' ).css({
                                         clip: 'auto',
@@ -68 +74 @@
                                         color: to
                                 });
+                                // Add class for different logo styles if title and description are visible.
                                 $( 'body' ).removeClass( 'title-tagline-hidden' );
                         }

branches/4.7/src/wp-content/themes/twentyseventeen/inc/custom-header.php

@@ -70 +70 @@
         // If we get this far, we have custom styles. Let's do this.
         ?>
-        <style type="text/css">
+        <style id="twentyseventeen-custom-header-styles" type="text/css">
         <?php
                 // Has the text been hidden?