From b0581285337c7574afb6bce0ce33028cf42e96a3 Mon Sep 17 00:00:00 2001
From: Silvan Verhoeven <silvan.verhoeven@student.hpi.de>
Date: Mon, 29 Jul 2024 20:48:18 +0200
Subject: [PATCH] feat: hide toc button if toc empty

---
 .../core/templates/core/information_page.html |  1 +
 myhpi/core/templates/core/minutes.html        |  5 +-
 myhpi/core/templates/core/sidebar.html        | 48 ++++++++++---------
 myhpi/core/templates/core/toc_button.html     | 15 ++++++
 myhpi/core/templatetags/core_extras.py        |  5 ++
 myhpi/templates/base.html                     |  8 ----
 6 files changed, 50 insertions(+), 32 deletions(-)
 create mode 100644 myhpi/core/templates/core/toc_button.html

diff --git a/myhpi/core/templates/core/information_page.html b/myhpi/core/templates/core/information_page.html
index 7c11e5a6..dc34609a 100644
--- a/myhpi/core/templates/core/information_page.html
+++ b/myhpi/core/templates/core/information_page.html
@@ -5,6 +5,7 @@
 
 {% block content %}
     {% with page.body|markdown as parsed_md %}
+        {% include "core/toc_button.html" with toc=parsed_md.1 %}
         <div class="row flex-column flex-lg-row">
             <div class="col-lg-9">
                 <h1 class="page-title">
diff --git a/myhpi/core/templates/core/minutes.html b/myhpi/core/templates/core/minutes.html
index 4a3f00c6..6380b5d4 100644
--- a/myhpi/core/templates/core/minutes.html
+++ b/myhpi/core/templates/core/minutes.html
@@ -6,8 +6,9 @@
 {% load bootstrap_icons %}
 
 {% block content %}
-    <div class="row flex-column flex-lg-row">
-        {% with page.body|markdown as parsed_md %}
+    {% with page.body|markdown as parsed_md %}
+        {% include "core/toc_button.html" with toc=parsed_md.1 %}
+        <div class="row flex-column flex-lg-row">
             <h1 class="page-title">
                 <span class="underline">
                     {{ page.title }}
diff --git a/myhpi/core/templates/core/sidebar.html b/myhpi/core/templates/core/sidebar.html
index 7e601742..0dc68b11 100644
--- a/myhpi/core/templates/core/sidebar.html
+++ b/myhpi/core/templates/core/sidebar.html
@@ -20,10 +20,12 @@ <h1 class="side-panel-title">{% translate "Visibility" %}</h1>
 <!-- Bootstrap 5.2 supports responsive offcanvas.
     This duplication is a workaround and should be removed after the upgrade -->
 <div class="d-none d-lg-block side-panel-container">
+    {%if parsed_md.1|hasTocContent %}
     <aside class="side-panel border-accent sidebar-container d-print-none">
         <h1 class="side-panel-title">{% translate "Table of contents" %}</h1>
         {{ parsed_md.1 }}
     </aside>
+    {% endif %}
     {% if page.attachments.all %}
         <aside class="side-panel border-accent sidebar-container d-print-none">
             <h1 class="side-panel-title">{% translate "Attachments" %}</h1>
@@ -38,28 +40,30 @@ <h1 class="side-panel-title">{% translate "Attachments" %}</h1>
     {% endif %}
 </div>
 
-<div class="offcanvas offcanvas-bottom" tabindex="-1" id="sidebar-offcanvas" aria-labelledby="sidebar-offcanvas-label">
-    <div class="offcanvas-header">
-        <h5 class="offcanvas-title" id="sidebar-offcanvas-label">{% translate "Table of contents" %}</h5>
-        <button type="button" class="btn-close" data-bs-dismiss="offcanvas"
-                aria-label="{% translate 'Close' %}"></button>
-    </div>
-    <div class="offcanvas-body">
-        <aside class="side-panel border-accent sidebar-container d-print-none">
-            <h1 class="side-panel-title">{% translate "Table of contents" %}</h1>
-            {{ parsed_md.1 }}
-        </aside>
-        {% if page.attachments.all %}
+{%if parsed_md.1|hasTocContent %}
+    <div class="offcanvas offcanvas-bottom" tabindex="-1" id="sidebar-offcanvas" aria-labelledby="sidebar-offcanvas-label">
+        <div class="offcanvas-header">
+            <h5 class="offcanvas-title" id="sidebar-offcanvas-label">{% translate "Table of contents" %}</h5>
+            <button type="button" class="btn-close" data-bs-dismiss="offcanvas"
+                    aria-label="{% translate 'Close' %}"></button>
+        </div>
+        <div class="offcanvas-body">
             <aside class="side-panel border-accent sidebar-container d-print-none">
-                <h1 class="side-panel-title">{% translate "Attachments" %}</h1>
-                <ul>
-                    {% for attachment in page.attachments.all %}
-                        <li>
-                            <a href="{{ attachment.url }}">{{ attachment.title }}</a>
-                        </li>
-                    {% endfor %}
-                </ul>
+                <h1 class="side-panel-title">{% translate "Table of contents" %}</h1>
+                {{ parsed_md.1 }}
             </aside>
-        {% endif %}
+            {% if page.attachments.all %}
+                <aside class="side-panel border-accent sidebar-container d-print-none">
+                    <h1 class="side-panel-title">{% translate "Attachments" %}</h1>
+                    <ul>
+                        {% for attachment in page.attachments.all %}
+                            <li>
+                                <a href="{{ attachment.url }}">{{ attachment.title }}</a>
+                            </li>
+                        {% endfor %}
+                    </ul>
+                </aside>
+            {% endif %}
+        </div>
     </div>
-</div>
\ No newline at end of file
+{% endif %}
\ No newline at end of file
diff --git a/myhpi/core/templates/core/toc_button.html b/myhpi/core/templates/core/toc_button.html
new file mode 100644
index 00000000..c003f882
--- /dev/null
+++ b/myhpi/core/templates/core/toc_button.html
@@ -0,0 +1,15 @@
+<!-- usage requires Table of Contents HTML in include context: https://docs.djangoproject.com/en/5.0/ref/templates/builtins/#include -->
+
+{% load core_extras %}
+{% load i18n %}
+{% load bootstrap_icons %}
+
+{%if toc|hasTocContent %}
+    <div id="sidebar-toggle" class="d-grid d-lg-none d-print-none">
+        <button type="button" data-bs-toggle="offcanvas" data-bs-target="#sidebar-offcanvas"
+            aria-controls="sidebar-offcanvas" class="btn btn-light mb-3" aria-label="{% translate 'Table of contents' %}">
+            {% bs_icon 'list-ol' size='1.3rem' %}
+            {% translate "Table of contents" %}
+        </button>
+    </div>
+{% endif %}
\ No newline at end of file
diff --git a/myhpi/core/templatetags/core_extras.py b/myhpi/core/templatetags/core_extras.py
index 1f88a659..2ca5cb17 100644
--- a/myhpi/core/templatetags/core_extras.py
+++ b/myhpi/core/templatetags/core_extras.py
@@ -35,6 +35,11 @@ def insert_footer(page):
     return {"footer_columns": [footer.column_1, footer.column_2, footer.column_3], "page": page}
 
 
+@register.filter()
+def hasTocContent(toc):
+    return "li" in toc
+
+
 @register.filter(name="tag_external_links")
 def tag_external_links(content):
     """Takes the content of a website and inserts external link icons after every external link."""
diff --git a/myhpi/templates/base.html b/myhpi/templates/base.html
index de63216a..951e01e2 100644
--- a/myhpi/templates/base.html
+++ b/myhpi/templates/base.html
@@ -150,14 +150,6 @@
                     </nav>
                 {% endif %}
 
-                <div id="sidebar-toggle" class="d-grid d-lg-none d-print-none">
-                    <button type="button" data-bs-toggle="offcanvas" data-bs-target="#sidebar-offcanvas"
-                            aria-controls="sidebar-offcanvas" class="btn btn-light mb-3" aria-label="{% translate 'Table of contents' %}">
-                        {% bs_icon 'list-ol' size='1.3rem' %}
-                        {% translate "Table of contents" %}
-                    </button>
-                </div>
-
                 {% block content %}{% endblock %}
             </div>
         </div>