Changeset 30394 for trunk/src/wp-content/themes/twentyfifteen/sidebar.php

Timestamp:

11/19/2014 08:27:09 PM (12 years ago)

Author:

iandstewart

Message:

Twenty Fifteen: doc cleanup, addresses removing periods from file header summaries, some spacing, loop documentation, and https instead of http.

Props DrewAPicture, MikeHansenMe, see #30149.

File:

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

trunk/src/wp-content/themes/twentyfifteen/sidebar.php

@@ -1 +1 @@
 <?php
 /**
- * The sidebar containing the main widget area.
+ * The sidebar containing the main widget area
  *
  * @package WordPress
@@ -10 +10 @@
 if ( has_nav_menu( 'primary' ) || has_nav_menu( 'social' ) || is_active_sidebar( 'sidebar-1' )  ) : ?>
         <div id="secondary" class="secondary">
+
                 <?php if ( has_nav_menu( 'primary' ) ) : ?>
-                <nav id="site-navigation" class="main-navigation" role="navigation">
-                        <?php
-                                wp_nav_menu( array(
-                                        'menu_class'     => 'nav-menu',
-                                        'theme_location' => 'primary',
-                                ) );
-                        ?>
-                </nav><!-- .main-navigation -->
+                        <nav id="site-navigation" class="main-navigation" role="navigation">
+                                <?php
+                                        // Primary navigation menu.
+                                        wp_nav_menu( array(
+                                                'menu_class'     => 'nav-menu',
+                                                'theme_location' => 'primary',
+                                        ) );
+                                ?>
+                        </nav><!-- .main-navigation -->
                 <?php endif; ?>
 
                 <?php if ( has_nav_menu( 'social' ) ) : ?>
-                <nav id="social-navigation" class="social-navigation" role="navigation">
-                        <?php
-                                wp_nav_menu( array(
-                                        'theme_location' => 'social',
-                                        'depth'          => 1,
-                                        'link_before'    => '<span class="screen-reader-text">',
-                                        'link_after'     => '</span>',
-                                ) );
-                        ?>
-                </nav><!-- .social-navigation -->
+                        <nav id="social-navigation" class="social-navigation" role="navigation">
+                                <?php
+                                        // Social links navigation menu.
+                                        wp_nav_menu( array(
+                                                'theme_location' => 'social',
+                                                'depth'          => 1,
+                                                'link_before'    => '<span class="screen-reader-text">',
+                                                'link_after'     => '</span>',
+                                        ) );
+                                ?>
+                        </nav><!-- .social-navigation -->
                 <?php endif; ?>
 
                 <?php if ( is_active_sidebar( 'sidebar-1' ) ) : ?>
-                <div id="widget-area" class="widget-area" role="complementary">
-                        <?php dynamic_sidebar( 'sidebar-1' ); ?>
-                </div><!-- .widget-area -->
+                        <div id="widget-area" class="widget-area" role="complementary">
+                                <?php dynamic_sidebar( 'sidebar-1' ); ?>
+                        </div><!-- .widget-area -->
                 <?php endif; ?>
+
         </div><!-- .secondary -->
+
 <?php endif; ?>