Changeset 23711

Timestamp:

03/15/2013 04:42:10 PM (11 years ago)

Author:

lancewillett

Message:

Twenty Twelve: fix regression introduced in r23572, allow hide/show of Header Text in Customizer. Fixes #23600.

Also remove another !important -- and there was much rejoicing.

Props SergeyBiryukov and obenland.

Location:

trunk/wp-content/themes/twentytwelve

Files:

• inc/custom-header.php (modified) (1 diff)
• js/theme-customizer.js (modified) (1 diff)

trunk/wp-content/themes/twentytwelve/inc/custom-header.php

@@ -70 +70 @@
         .site-title,
         .site-description {
-            position: absolute !important;
+            position: absolute;
             clip: rect(1px 1px 1px 1px); /* IE7 */
             clip: rect(1px, 1px, 1px, 1px);

trunk/wp-content/themes/twentytwelve/js/theme-customizer.js

@@ -18 +18 @@
         } );
     } );
+
     // Header text color
     wp.customize( 'header_textcolor', function( value ) {
         value.bind( function( to ) {
-            $( '.site-title a, .site-description' ).css( 'color', to );
+            if ( 'blank' === to ) {
+                $( '.site-title, .site-title a, .site-description' ).css( {
+                    'clip': 'rect(1px, 1px, 1px, 1px)',
+                    'position': 'absolute'
+                } );
+            } else {
+                $( '.site-title, .site-title a, .site-description' ).css( {
+                    'clip': 'auto',
+                    'color': to,
+                    'position': 'relative'
+                } );
+            }
         } );
     } );