UX: hide column containers if empty (#21)

discourse · Oct 3, 2023 · 333f922 · 333f922
1 parent 26ecbdf
commit 333f922
Showing 1 changed file with 17 additions and 13 deletions.
diff --git a/javascripts/discourse/components/versatile-banner-column.hbs b/javascripts/discourse/components/versatile-banner-column.hbs
@@ -1,14 +1,18 @@
-<div class="{{@columnClass}} single-box">
-  {{#if @icon}}
-    <div class="icon">
-      {{#if this.isHttpLink}}
-        <img class="responsive-img" src={{@icon}} alt="Icon" />
-      {{else}}
-        {{d-icon @icon}}
-      {{/if}}
-    </div>
-  {{/if}}
-  <div>
-    {{html-safe @columnContent}}
+{{#if (or @icon @columnContent)}}
+  <div class="{{@columnClass}} single-box">
+    {{#if @icon}}
+      <div class="icon">
+        {{#if this.isHttpLink}}
+          <img class="responsive-img" src={{@icon}} alt="Icon" />
+        {{else}}
+          {{d-icon @icon}}
+        {{/if}}
+      </div>
+    {{/if}}
+    {{#if @columnContent}}
+      <div>
+        {{html-safe @columnContent}}
+      </div>
+    {{/if}}
   </div>
-</div>
+{{/if}}