From 1a7f385ccc4749eca6176e7f2b710ed838f4f162 Mon Sep 17 00:00:00 2001
From: Deimos <deimos@tildes.net>
Date: Sat, 15 Dec 2018 18:38:07 -0700
Subject: [PATCH] Update "no topics found" message

Previously, the message would always tell users to try choosing a longer
time period, even if they were already on "all time".
---
 tildes/tildes/templates/topic_listing.jinja2 | 18 +++++++++++-------
 1 file changed, 11 insertions(+), 7 deletions(-)

diff --git a/tildes/tildes/templates/topic_listing.jinja2 b/tildes/tildes/templates/topic_listing.jinja2
index 5d68ac1..3bc57b4 100644
--- a/tildes/tildes/templates/topic_listing.jinja2
+++ b/tildes/tildes/templates/topic_listing.jinja2
@@ -115,14 +115,18 @@
 
 {% if not topics %}
   <div class="empty">
-    <h2 class="empty-title">No topics in the selected time period</h2>
-    {% if is_single_group and request.has_permission('post_topic', group) %}
-      <p class="empty-subtitle">Choose a longer time period, or break the silence by posting one yourself.</p>
-      <div class="empty-action">
-        <a href="/~{{ group.path }}/new_topic" class="btn btn-primary">Post a new topic</a>
-      </div>
+    {% if period %}
+      <h2 class="empty-title">No topics in the selected time period</h2>
+      {% if is_single_group and request.has_permission('post_topic', group) %}
+        <p class="empty-subtitle">Try choosing a longer time period, or break the silence by posting one yourself.</p>
+        <div class="empty-action">
+          <a href="/~{{ group.path }}/new_topic" class="btn btn-primary">Post a new topic</a>
+        </div>
+      {% else %}
+        <p class="empty-subtitle">Try choosing a longer time period to find some.</p>
+      {% endif %}
     {% else %}
-      <p class="empty-subtitle">Choose a longer time period to find some.</p>
+      <h2 class="empty-title">No topics found</h2>
     {% endif %}
   </div>
 {% endif %}