Context Navigation

Ticket #25389: 25389.2.diff

File 25389.2.diff, 1.3 KB (added by taupecat, 11 years ago)
Supercedes 25389.diff; tests for presence of footer widgets and only applies min-height if no footer widgets exist.

diff --git wp-content/themes/twentyfourteen/functions.php wp-content/themes/twentyfourteen/functions.php
index 28ef45c..d70a8f3 100644

  add_action( 'pre_get_posts', 'twentyfourteen_pre_get_posts' );
  * 1. Single or multiple authors.
  * 2. Index views.
  * 3. Full-width content layout.
+ * 4. Absence of footer widgets.
+ *
  * @param array $classes A list of existing body class values.
  * @return array The filtered body class list.
-…
+ function twentyfourteen_body_classes( $classes ) {
                 || is_attachment() )
                 $classes[] = 'full-width';
+        if ( ! is_active_sidebar( 'sidebar-4' ) )
+                $classes[] = 'no-footer-widgets';
         return $classes;
+}
 add_filter( 'body_class', 'twentyfourteen_body_classes' );

diff --git wp-content/themes/twentyfourteen/style.css wp-content/themes/twentyfourteen/style.css
index a42f93a..e055c0d 100644

  span > object {
                 padding: 0 27px;
                 width: 128px;
+        }
+        body.no-footer-widgets #secondary {
+                min-height: 100vh;
+        }
         .site-description {
                 display: block;
                 line-height: 1.6363636363;