Changeset 14708 for trunk/wp-content/themes/twentyten/sidebar.php

Timestamp:

05/17/2010 06:36:11 AM (15 years ago)

Author:

nacin

Message:

In Twenty Ten, documentation and code improvements. see #12695.

File:

• trunk/wp-content/themes/twentyten/sidebar.php (modified) (4 diffs)

trunk/wp-content/themes/twentyten/sidebar.php

@@ -5 +5 @@
  * @package WordPress
  * @subpackage Twenty_Ten
- * @since 3.0.0
+ * @since Twenty Ten 1.0
  */
 ?>
@@ -11 +11 @@
         <div id="primary" class="widget-area" role="complementary">
             <ul class="xoxo">
-<?php if ( ! dynamic_sidebar( 'primary-widget-area' ) ) : // begin primary widget area ?>
+
+<?php
+    /* When we call the dynamic_sidebar() function, it'll spit out
+     * the widgets for that widget area. If it instead returns false,
+     * then the sidebar simply doesn't exist, so we'll hard-code in
+     * some default sidebar stuff just in case.
+     */
+    if ( ! dynamic_sidebar( 'primary-widget-area' ) ) : ?>
+    
             <li id="search" class="widget-container widget_search">
                 <?php get_search_form(); ?>
@@ -31 +39 @@
                 </ul>
             </li>
-<?php endif; // end primary widget area ?>
+
+        <?php endif; // end primary widget area ?>
             </ul>
         </div><!-- #primary .widget-area -->
 
-<?php if ( is_active_sidebar( 'secondary-widget-area' ) ) : // Nothing here by default and design ?>
+<?php
+    // A second sidebar for widgets, just because.
+    if ( is_active_sidebar( 'secondary-widget-area' ) ) : ?>
+
         <div id="secondary" class="widget-area" role="complementary">
             <ul class="xoxo">
@@ -41 +53 @@
             </ul>
         </div><!-- #secondary .widget-area -->
+
 <?php endif; ?>