Changeset 21555 for trunk/wp-content/themes/twentytwelve/sidebar-home.php

Timestamp:

08/20/2012 06:45:00 PM (12 years ago)

Author:

lancewillett

Message:

Twenty Twelve: change homepage widget areas to use columns or multiple areas instead of one big area. Fixes #21619, props obenland.

File:

• trunk/wp-content/themes/twentytwelve/sidebar-home.php (modified) (2 diffs)

trunk/wp-content/themes/twentytwelve/sidebar-home.php

@@ -1 +1 @@
 <?php
 /**
- * The Sidebar containing the homepage widget area.
+ * The Sidebar containing the homepage widget areas.
  *
- * If no active widgets in this sidebar, it will be hidden completely.
+ * If no active widgets in either sidebar, they will be hidden completely.
  *
  * @package WordPress
@@ -9 +9 @@
  * @since Twenty Twelve 1.0
  */
+
+/*
+The homepage widget area is triggered if any of the areas
+have widgets. So let's check that first.
+
+If none of the sidebars have widgets, then let's bail early.
+*/
+if ( ! is_active_sidebar( 2 ) && ! is_active_sidebar( 3 ) )
+    return;
+
+// If we get this far, we have widgets. Let do this.
 ?>
+<div id="secondary" <?php twentytwelve_homepage_sidebar_class(); ?> role="complementary">
+    <?php if ( is_active_sidebar( 2 ) ) : ?>
+    <div class="first home-widgets">
+        <?php dynamic_sidebar( 'sidebar-2' ); ?>
+    </div><!-- .first .home-widgets -->
+    <?php endif; ?>
 
-    <?php if ( is_active_sidebar( 'sidebar-home' ) ) : ?>
-        <div id="secondary" class="widget-area" role="complementary">
-            <?php dynamic_sidebar( 'sidebar-home' ); ?>
-        </div><!-- #secondary .widget-area -->
+    <?php if ( is_active_sidebar( 3 ) ) : ?>
+    <div class="second home-widgets">
+        <?php dynamic_sidebar( 'sidebar-3' ); ?>
+    </div><!-- .second .home-widgets -->
     <?php endif; ?>
+</div><!-- #secondary .widget-area -->