hide header if app has no interactive elements (bug 955828)

mozilla · Jan 3, 2014 · 277347f · cvan · Jan 3, 2014 · 277347f
1 parent 3617699
commit 277347f
Showing 1 changed file with 16 additions and 14 deletions.
diff --git a/mkt/developers/templates/developers/apps/ratings/ratings_summary.html b/mkt/developers/templates/developers/apps/ratings/ratings_summary.html
@@ -53,23 +53,25 @@ <h3>{{ _('Content Ratings') }}</h3>
         </tbody>
       </table>
 
-      <h3>{{ _('Interactive Elements') }}</h3>
-      <div class="interactive-elements c">
-        {% set interactives = addon.get_interactives() %}
-        <div class="names">
+      {% set interactives = addon.get_interactives() %}
+      {% if interactives %}
+        <h3>{{ _('Interactive Elements') }}</h3>
+        <div class="interactive-elements c">
+          <div class="names">
+            {% for interactive in interactives %}
+              <span class="dot-sep">{{ interactive.name }}</span>
+            {% endfor %}
+          </div>
+          {% set inter_icons = mkt.ratingsbodies.IARC_ICONS.interactive_elements %}
           {% for interactive in interactives %}
-            <span class="dot-sep">{{ interactive.name }}</span>
+            {% set inter_icon = inter_icons[interactive.label] %}
+            {% if inter_icon %}
+              <img src="{{ media(inter_icon) }}"
+                   class="icon {{ interactive.label }}" title="{{ interactive.name }}">
+            {% endif %}
           {% endfor %}
         </div>
-        {% set inter_icons = mkt.ratingsbodies.IARC_ICONS.interactive_elements %}
-        {% for interactive in interactives %}
-          {% set inter_icon = inter_icons[interactive.label] %}
-          {% if inter_icon %}
-            <img src="{{ media(inter_icon) }}"
-                 class="icon {{ interactive.label }}" title="{{ interactive.name }}">
-          {% endif %}
-        {% endfor %}
-      </div>
+      {% endif %}
     </div>
   </section>