Merge pull request #1864 from IanDelMar/html-comments

Fix HTML comments

404.php

@@ -76,7 +76,7 @@ $container = get_theme_mod( 'understrap_container_type' );
 
 					</section><!-- .error-404 -->
 
-				</main><!-- #main -->
+				</main>
 
 			</div><!-- #primary -->
 

archive.php

@@ -21,8 +21,10 @@ $container = get_theme_mod( 'understrap_container_type' );
 
 		<div class="row">
 
-			<!-- Do the left sidebar check -->
-			<?php get_template_part( 'global-templates/left-sidebar-check' ); ?>
+			<?php
+			// Do the left sidebar check and open div#primary.
+			get_template_part( 'global-templates/left-sidebar-check' );
+			?>
 
 			<main class="site-main" id="main">
 
@@ -52,12 +54,13 @@ $container = get_theme_mod( 'understrap_container_type' );
 				}
 				?>
 
-			</main><!-- #main -->
+			</main>
 
 			<?php
 			// Display the pagination component.
 			understrap_pagination();
-			// Do the right sidebar check.
+
+			// Do the right sidebar check and close div#primary.
 			get_template_part( 'global-templates/right-sidebar-check' );
 			?>
 

author.php

@@ -20,8 +20,10 @@ $container = get_theme_mod( 'understrap_container_type' );
 
 		<div class="row">
 
-			<!-- Do the left sidebar check -->
-			<?php get_template_part( 'global-templates/left-sidebar-check' ); ?>
+			<?php
+			// Do the left sidebar check and open div#primary.
+			get_template_part( 'global-templates/left-sidebar-check' );
+			?>
 
 			<main class="site-main" id="main">
 
@@ -82,8 +84,8 @@ $container = get_theme_mod( 'understrap_container_type' );
 
 				</header><!-- .page-header -->
 
-				<!-- The Loop -->
 				<?php
+				// Start the loop.
 				if ( have_posts() ) {
 					while ( have_posts() ) {
 						the_post();
@@ -93,15 +95,17 @@ $container = get_theme_mod( 'understrap_container_type' );
 					get_template_part( 'loop-templates/content', 'none' );
 				}
 				?>
-				<!-- End Loop -->
 
-			</main><!-- #main -->
+			</main>
 
-			<!-- The pagination component -->
-			<?php understrap_pagination(); ?>
 
-			<!-- Do the right sidebar check -->
-			<?php get_template_part( 'global-templates/right-sidebar-check' ); ?>
+			<?php
+			// Display the pagination component.
+			understrap_pagination();
+
+			// Do the right sidebar check and close div#primary.
+			get_template_part( 'global-templates/right-sidebar-check' );
+			?>
 
 		</div> <!-- .row -->
 

footer.php

@@ -33,15 +33,16 @@ $container = get_theme_mod( 'understrap_container_type' );
 
 				</footer><!-- #colophon -->
 
-			</div><!--col end -->
+			</div><!-- col -->
 
-		</div><!-- row end -->
+		</div><!-- .row -->
 
-	</div><!-- container end -->
+	</div><!-- .container(-fluid) -->
 
-</div><!-- wrapper end -->
+</div><!-- #wrapper-footer -->
 
-</div><!-- #page we need this extra closing tag here -->
+<?php // Closing div#page from header.php. ?>
+</div><!-- #page -->
 
 <?php wp_footer(); ?>
 

global-templates/navbar-collapse-bootstrap4.php

@@ -48,4 +48,4 @@ $container = get_theme_mod( 'understrap_container_type' );
 	</div><!-- .container -->
 <?php endif; ?>
 
-</nav><!-- .site-navigation -->
+</nav><!-- #main-nav -->

global-templates/navbar-collapse-bootstrap5.php

@@ -45,4 +45,4 @@ $container = get_theme_mod( 'understrap_container_type' );
 
 	</div><!-- .container(-fluid) -->
 
-</nav><!-- .site-navigation -->
+</nav><!-- #main-nav -->

global-templates/navbar-offcanvas-bootstrap5.php

@@ -52,4 +52,4 @@ $container = get_theme_mod( 'understrap_container_type' );
 
 	</div><!-- .container(-fluid) -->
 
-</nav><!-- .site-navigation -->
+</nav><!-- #main-nav -->

global-templates/right-sidebar-check.php

@@ -7,9 +7,10 @@
 
 // Exit if accessed directly.
 defined( 'ABSPATH' ) || exit;
-?>
 
-</div><!-- #closing the primary container from /global-templates/left-sidebar-check.php -->
+// Closing the primary container from /global-templates/left-sidebar-check.php.
+?>
+</div><!-- #primary -->
 
 <?php
 $sidebar_pos = get_theme_mod( 'understrap_sidebar_position' );

header.php

@@ -33,4 +33,4 @@ $navbar_type       = get_theme_mod( 'understrap_navbar_type', 'collapse' );
 
 		<?php get_template_part( 'global-templates/navbar', $navbar_type . '-' . $bootstrap_version ); ?>
 
-	</header><!-- #wrapper-navbar end -->
+	</header><!-- #wrapper-navbar -->

inc/pagination.php

@@ -71,6 +71,7 @@ if ( ! function_exists( 'understrap_pagination' ) ) {
 		}
 		?>
 
+		<!-- The pagination component -->
 		<nav aria-labelledby="posts-nav-label">
 
 			<h2 id="posts-nav-label" class="screen-reader-text">

inc/template-tags.php

@@ -297,7 +297,7 @@ if ( ! function_exists( 'understrap_post_nav' ) ) {
 				}
 				?>
 			</div><!-- .nav-links -->
-		</nav><!-- .navigation -->
+		</nav><!-- .post-navigation -->
 		<?php
 	}
 }

inc/woocommerce.php

@@ -58,11 +58,11 @@ if ( ! function_exists( 'understrap_woocommerce_wrapper_end' ) ) {
 	 * Display the theme specific end of the page wrapper.
 	 */
 	function understrap_woocommerce_wrapper_end() {
-		echo '</main><!-- #main -->';
+		echo '</main>';
 		get_template_part( 'global-templates/right-sidebar-check' );
 		echo '</div><!-- .row -->';
-		echo '</div><!-- Container end -->';
-		echo '</div><!-- Wrapper end -->';
+		echo '</div><!-- .container(-fluid) -->';
+		echo '</div><!-- #woocommerce-wrapper -->';
 	}
 }
 

index.php

@@ -29,8 +29,10 @@ $container = get_theme_mod( 'understrap_container_type' );
 
 		<div class="row">
 
-			<!-- Do the left sidebar check and opens the primary div -->
-			<?php get_template_part( 'global-templates/left-sidebar-check' ); ?>
+			<?php
+			// Do the left sidebar check and open div#primary.
+			get_template_part( 'global-templates/left-sidebar-check' );
+			?>
 
 			<main class="site-main" id="main">
 
@@ -52,13 +54,15 @@ $container = get_theme_mod( 'understrap_container_type' );
 				}
 				?>
 
-			</main><!-- #main -->
+			</main>
 
-			<!-- The pagination component -->
-			<?php understrap_pagination(); ?>
+			<?php
+			// Display the pagination component.
+			understrap_pagination();
 
-			<!-- Do the right sidebar check -->
-			<?php get_template_part( 'global-templates/right-sidebar-check' ); ?>
+			// Do the right sidebar check and close div#primary.
+			get_template_part( 'global-templates/right-sidebar-check' );
+			?>
 
 		</div><!-- .row -->
 

loop-templates/content-author.php

@@ -44,4 +44,4 @@ defined( 'ABSPATH' ) || exit;
 
 	</footer><!-- .entry-footer -->
 
-</article><!-- #post-## -->
+</article><!-- #post-<?php the_ID(); ?> -->

loop-templates/content-page.php

@@ -37,4 +37,4 @@ defined( 'ABSPATH' ) || exit;
 
 	</footer><!-- .entry-footer -->
 
-</article><!-- #post-## -->
+</article><!-- #post-<?php the_ID(); ?> -->

loop-templates/content-search.php

@@ -44,4 +44,4 @@ defined( 'ABSPATH' ) || exit;
 
 	</footer><!-- .entry-footer -->
 
-</article><!-- #post-## -->
+</article><!-- #post-<?php the_ID(); ?> -->

loop-templates/content-single.php

@@ -40,4 +40,4 @@ defined( 'ABSPATH' ) || exit;
 
 	</footer><!-- .entry-footer -->
 
-</article><!-- #post-## -->
+</article><!-- #post-<?php the_ID(); ?> -->

loop-templates/content.php

@@ -47,4 +47,4 @@ defined( 'ABSPATH' ) || exit;
 
 	</footer><!-- .entry-footer -->
 
-</article><!-- #post-## -->
+</article><!-- #post-<?php the_ID(); ?> -->

page-templates/both-sidebarspage.php

@@ -48,7 +48,7 @@ $container = get_theme_mod( 'understrap_container_type' );
 					}
 					?>
 
-				</main><!-- #main -->
+				</main>
 
 			</div><!-- #primary -->
 

page-templates/fullwidthpage.php

@@ -45,11 +45,11 @@ if ( is_page_template( 'page-templates/no-title.php' ) ) {
 					}
 					?>
 
-				</main><!-- #main -->
+				</main>
 
 			</div><!-- #primary -->
 
-		</div><!-- .row end -->
+		</div><!-- .row -->
 
 	</div><!-- #content -->
 

page-templates/left-sidebarpage.php

@@ -39,7 +39,7 @@ $container = get_theme_mod( 'understrap_container_type' );
 					}
 					?>
 
-				</main><!-- #main -->
+				</main>
 
 			</div><!-- #primary -->
 

page-templates/right-sidebarpage.php

@@ -37,7 +37,7 @@ $container = get_theme_mod( 'understrap_container_type' );
 					}
 					?>
 
-				</main><!-- #main -->
+				</main>
 
 			</div><!-- #primary -->
 

page.php

@@ -25,8 +25,10 @@ $container = get_theme_mod( 'understrap_container_type' );
 
 		<div class="row">
 
-			<!-- Do the left sidebar check -->
-			<?php get_template_part( 'global-templates/left-sidebar-check' ); ?>
+			<?php
+			// Do the left sidebar check and open div#primary.
+			get_template_part( 'global-templates/left-sidebar-check' );
+			?>
 
 			<main class="site-main" id="main">
 
@@ -42,10 +44,12 @@ $container = get_theme_mod( 'understrap_container_type' );
 				}
 				?>
 
-			</main><!-- #main -->
+			</main>
 
-			<!-- Do the right sidebar check -->
-			<?php get_template_part( 'global-templates/right-sidebar-check' ); ?>
+			<?php
+			// Do the right sidebar check and close div#primary.
+			get_template_part( 'global-templates/right-sidebar-check' );
+			?>
 
 		</div><!-- .row -->
 

search.php

@@ -20,8 +20,10 @@ $container = get_theme_mod( 'understrap_container_type' );
 
 		<div class="row">
 
-			<!-- Do the left sidebar check and opens the primary div -->
-			<?php get_template_part( 'global-templates/left-sidebar-check' ); ?>
+			<?php
+			// Do the left sidebar check and open div#primary.
+			get_template_part( 'global-templates/left-sidebar-check' );
+			?>
 
 			<main class="site-main" id="main">
 
@@ -61,13 +63,15 @@ $container = get_theme_mod( 'understrap_container_type' );
 
 				<?php endif; ?>
 
-			</main><!-- #main -->
+			</main>
 
-			<!-- The pagination component -->
-			<?php understrap_pagination(); ?>
+			<?php
+			// Display the pagination component.
+			understrap_pagination();
 
-			<!-- Do the right sidebar check -->
-			<?php get_template_part( 'global-templates/right-sidebar-check' ); ?>
+			// Do the right sidebar check and close div#primary.
+			get_template_part( 'global-templates/right-sidebar-check' );
+			?>
 
 		</div><!-- .row -->
 

single.php

@@ -18,8 +18,10 @@ $container = get_theme_mod( 'understrap_container_type' );
 
 		<div class="row">
 
-			<!-- Do the left sidebar check -->
-			<?php get_template_part( 'global-templates/left-sidebar-check' ); ?>
+			<?php
+			// Do the left sidebar check and open div#primary.
+			get_template_part( 'global-templates/left-sidebar-check' );
+			?>
 
 			<main class="site-main" id="main">
 
@@ -36,10 +38,12 @@ $container = get_theme_mod( 'understrap_container_type' );
 				}
 				?>
 
-			</main><!-- #main -->
+			</main>
 
-			<!-- Do the right sidebar check -->
-			<?php get_template_part( 'global-templates/right-sidebar-check' ); ?>
+			<?php
+			// Do the right sidebar check and close div#primary.
+			get_template_part( 'global-templates/right-sidebar-check' );
+			?>
 
 		</div><!-- .row -->