CRO-314: also includes CRO-179. Hide brands in the footer if

there are no brands, hide Info section in the footer if address isn't provided
bigcommerce · Jul 26, 2017 · bf8d1c6 · bf8d1c6
1 parent 5577d65
commit bf8d1c6
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/templates/components/common/footer.html b/templates/components/common/footer.html
@@ -42,7 +42,7 @@ <h5 class="footer-info-heading">{{lang 'footer.categories'}}</h5>
                 </ul>
             </article>
 
-            {{#if theme_settings.shop_by_brand_show_footer}}
+            {{#and theme_settings.shop_by_brand_show_footer shop_by_brand.length}}
             <article class="footer-info-col footer-info-col--small" data-section-type="footer-brands">
                 <h5 class="footer-info-heading">{{lang 'footer.brands'}}</h5>
                 <ul class="footer-info-list">
@@ -54,15 +54,17 @@ <h5 class="footer-info-heading">{{lang 'footer.brands'}}</h5>
                     <li><a href="{{urls.brands}}">{{lang 'common.view_all'}}</a></li>
                 </ul>
             </article>
-            {{/if}}
+            {{/and}}
 
+            {{#if settings.address}}
             <article class="footer-info-col footer-info-col--small" data-section-type="storeInfo">
                 <h5 class="footer-info-heading">{{lang 'footer.info'}}</h5>
                 <address>{{nl2br settings.address}}</address>
                 {{#if settings.phone_number}}
                     <strong>{{lang 'footer.call_us' phone_number=settings.phone_number}}</strong>
                 {{/if}}
             </article>
+            {{/if}}
 
             <article class="footer-info-col" data-section-type="newsletterSubscription">
                 {{#if settings.show_newsletter_box}}