Changeset 30230 for trunk/src/wp-content/themes/twentyfifteen/inc/custom-header.php

Timestamp:

11/04/2014 06:41:56 PM (11 years ago)

Author:

iandstewart

Message:

Twenty Fifteen: Simplify the header, sidebar, background controls and make customization faster for users to do. This has the added benefit of fixing our bug where hidden header text wasn't being updated on color scheme switch. Nice.

Props celloexpressions, iamtakashi, fixes #30164 and #29980.

File:

• trunk/src/wp-content/themes/twentyfifteen/inc/custom-header.php (modified) (3 diffs)

trunk/src/wp-content/themes/twentyfifteen/inc/custom-header.php

@@ -80 +80 @@
 function twentyfifteen_header_style() {
     $header_image = get_header_image();
-    $text_color   = get_header_textcolor();
 
     // If no custom options for text are set, let's bail.
-    if ( empty( $header_image ) && $text_color == get_theme_support( 'custom-header', 'default-text-color' ) ) {
+    if ( empty( $header_image ) && display_header_text() ) {
         return;
     }
@@ -126 +125 @@
             position: absolute;
         }
-    <?php
-        // If the user has set a custom color for the text use that
-        elseif ( $text_color != get_theme_support( 'custom-header', 'default-text-color' ) ) :
-    ?>
-        .site-title a,
-        .site-title a:hover,
-        .site-title a:focus,
-        .site-description {
-            color: #<?php echo esc_attr( $text_color ); ?>;
-        }
     <?php endif; ?>
     </style>
@@ -229 +218 @@
     $css = '
         /* Custom Sidebar Text Color */
+        .site-title a,
+        .site-description,
         .secondary-toggle:before {
             color: %1$s;
+        }
+
+        .site-title a:hover,
+        .site-title a:focus {
+            color: %1$s; /* Fallback for IE7 and IE8 */
+            color: %2$s;
         }