avoid outputting an empty div here even when the ad zone is disabled

WPBuddy · Aug 25, 2016 · ecdc31c · ecdc31c
1 parent e1c30f5
commit ecdc31c
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 19 deletions.
diff --git a/header.php b/header.php
@@ -83,11 +83,14 @@
 
 	<div id="page" class="hfeed clearfix">
 
-		<?php get_template_part( 'partials/nav', 'sticky' ); ?>
-
-		<?php get_template_part('partials/header-ad-zone'); ?>
+		<?php 
+
+			get_template_part( 'partials/nav', 'sticky' ); 
+
+			if ( of_get_option( 'leaderboard_enabled' ) == TRUE ) {
+				get_template_part( 'partials/header-ad-zone' );
+			}
 
-		<?php
 			/**
 			 * Fires before the Largo header content.
 			 *
@@ -103,26 +106,21 @@
 			 * @since 0.4
 			 */
 			do_action( 'largo_after_header' );
-		?>
-
-		<?php
+
 			get_template_part( 'partials/nav', 'main' );
 
 			if ( SHOW_SECONDARY_NAV === TRUE ) {
 				get_template_part( 'partials/nav', 'secondary' );
 			}
-		?>
-
-		<?php get_template_part('partials/homepage-alert'); ?>
 
-		<?php
+			get_template_part('partials/homepage-alert'); 
 
-		/**
-		 * Fires after the Largo navigation content.
-		 *
-		 * @since 0.4
-		 */
-		do_action( 'largo_after_nav' );
+			/**
+			 * Fires after the Largo navigation content.
+			 *
+			 * @since 0.4
+			*/
+			do_action( 'largo_after_nav' );
 
 		?>
 

diff --git a/partials/header-ad-zone.php b/partials/header-ad-zone.php
@@ -1,8 +1,8 @@
 <?php
-if (is_active_sidebar('header-ads')) { ?>
+if ( is_active_sidebar( 'header-ads' ) ) { ?>
 <div class="header-ad-zone max-wide">
 	<div id="header-ad-zone-container">
-		<?php dynamic_sidebar('header-ads'); ?>
+		<?php dynamic_sidebar( 'header-ads' ); ?>
 	</div>
 </div>
 <?php } ?>