Changeset 25736

Timestamp:

10/08/2013 09:36:46 PM (12 years ago)

Author:

lancewillett

Message:

Twenty Fourteen: fill browser height to 100% for views with short content. Accounts for footer widgets, in which case the height isn't set to 100% so that the widgets remain visible. Props wycks and taupecat, fixes #25389.

Location:

trunk/src/wp-content/themes/twentyfourteen

Files:

• functions.php (modified) (2 diffs)
• style.css (modified) (1 diff)

trunk/src/wp-content/themes/twentyfourteen/functions.php

@@ -460 +460 @@
  * 2. Index views.
  * 3. Full-width content layout.
+ * 4. Presence of footer widgets.
  *
  * @param array $classes A list of existing body class values.
@@ -477 +478 @@
         $classes[] = 'full-width';
 
+    if ( is_active_sidebar( 'sidebar-4' ) )
+        $classes[] = 'footer-widgets';
+
     return $classes;
 }

trunk/src/wp-content/themes/twentyfourteen/style.css

@@ -2518 +2518 @@
         line-height: 1.6363636363;
         margin: 0 0 0 -100%;
+        min-height: 100vh;
         padding: 0 27px;
         width: 128px;
+    }
+    body.footer-widgets #secondary {
+        min-height: 0;
     }
     .site-description {