Changeset 33092

Timestamp:

07/06/2015 05:26:02 PM (11 years ago)

Author:

obenland

Message:

Twenty Fifteen: Display sidebar background color with Header Image.

If the header image has a transparent background, the sidebar background color
should be visible. With the background shorthand it could never be over set
on the front-end however.

Props valendesigns.
Fixes #31460.

File:

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

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

@@ -134 +134 @@
         ?>
                 .site-header {
-                        background: url(<?php header_image(); ?>) no-repeat 50% 50%;
+
+                        /*
+                         * No shorthand so the Customizer can override individual properties.
+                         * @see https://core.trac.wordpress.org/ticket/31460
+                         */
+                        background-image: url(<?php header_image(); ?>);
+                        background-repeat: no-repeat;
+                        background-position: 50% 50%;
                         -webkit-background-size: cover;
                         -moz-background-size:    cover;
@@ -143 +150 @@
                 @media screen and (min-width: 59.6875em) {
                         body:before {
-                                background: url(<?php header_image(); ?>) no-repeat 100% 50%;
+
+                                /*
+                                 * No shorthand so the Customizer can override individual properties.
+                                 * @see https://core.trac.wordpress.org/ticket/31460
+                                 */
+                                background-image: url(<?php header_image(); ?>);
+                                background-repeat: no-repeat;
+                                background-position: 100% 50%;
                                 -webkit-background-size: cover;
                                 -moz-background-size:    cover;