From 35ea0f43b48dbaad318998152d10a6b44d4488aa Mon Sep 17 00:00:00 2001
From: Ivan Fonseca <ivanfon@riseup.net>
Date: Wed, 1 Aug 2018 14:54:37 -0400
Subject: [PATCH] Hide topic text excerpt for spoilers

---
 tildes/tildes/templates/macros/topics.jinja2 | 30 +++++++++++++-------
 1 file changed, 20 insertions(+), 10 deletions(-)

diff --git a/tildes/tildes/templates/macros/topics.jinja2 b/tildes/tildes/templates/macros/topics.jinja2
index 5ad4112..4519847 100644
--- a/tildes/tildes/templates/macros/topics.jinja2
+++ b/tildes/tildes/templates/macros/topics.jinja2
@@ -47,20 +47,30 @@
     </div>
 
     {% if topic.is_text_type and topic.get_content_metadata('excerpt') %}
-      {# if the "excerpt" is the full text, don't wrap in <details> #}
-      {% if not topic.get_content_metadata('excerpt').endswith('...') %}
-        <p class="topic-text-excerpt">{{ topic.get_content_metadata('excerpt') }}</p>
-      {% else %}
-        <details class="topic-text-excerpt"
-          {% if request.user and request.user.open_new_tab_text %}
-          data-js-external-links-new-tabs
-          {% endif %}
-        >
+      {# warn about spoilers #}
+      {% if 'spoiler' in topic.tags %}
+        <details class="topic-text-excerpt">
           <summary>
-            <span>{{ topic.get_content_metadata('excerpt') }}</span>
+            <span>Warning: this post may contain spoilers.</span>
           </summary>
           {{ topic.rendered_html|safe }}
         </details>
+      {% else %}
+        {# if the "excerpt" is the full text, don't wrap in <details> #}
+        {% if not topic.get_content_metadata('excerpt').endswith('...') %}
+          <p class="topic-text-excerpt">{{ topic.get_content_metadata('excerpt') }}</p>
+        {% else %}
+          <details class="topic-text-excerpt"
+            {% if request.user and request.user.open_new_tab_text %}
+            data-js-external-links-new-tabs
+            {% endif %}
+          >
+            <summary>
+              <span>{{ topic.get_content_metadata('excerpt') }}</span>
+            </summary>
+            {{ topic.rendered_html|safe }}
+          </details>
+        {% endif %}
       {% endif %}
     {% endif %}