Only show ToC if there are enough items

website/blog/models.py

@@ -23,11 +23,8 @@ class BlogPostListPage(BaseListingPage):
     ]
 
     @cached_property
-    def table_of_contents(self) -> list[TocEntry]:
-        """
-        Showing an accurate ToC is complex alongside pagination
-        """
-        return []
+    def show_table_of_contents(self) -> bool:
+        return False
 
     def get_listing_pages(self) -> models.QuerySet:
         return (

website/common/models.py

@@ -96,6 +96,10 @@ class BaseContentPage(BasePage, MetadataMixin):
     def table_of_contents(self) -> list[TocEntry]:
         return get_table_of_contents(self.content_html)
 
+    @cached_property
+    def show_table_of_contents(self) -> bool:
+        return len(self.table_of_contents) >= 3
+
     @cached_property
     def reading_time(self) -> timedelta:
         """
@@ -204,8 +208,8 @@ class BaseListingPage(RoutablePageMixin, BaseContentPage):
         return context
 
     @cached_property
-    def table_of_contents(self) -> list[TocEntry]:
-        return []
+    def show_table_of_contents(self) -> bool:
+        return False
 
     @cached_property
     def show_reading_time(self) -> bool:

website/common/templates/wagtail_base.html

@@ -32,7 +32,7 @@
           </div>
           <div class="column is-narrow dropdown-wrapper is-hidden-touch is-grouped">
             {% block hero_buttons %}
-              {% if page.table_of_contents %}
+              {% if page.show_table_of_contents %}
                 <div class="dropdown is-hoverable is-right" id="table-of-contents">
                   <div class="dropdown-trigger">
                     <button class="button is-radiusless" aria-haspopup="true" aria-controls="toc-menu">

website/search/models.py

@@ -10,7 +10,7 @@ from wagtail.models import Page
 from wagtail.search.utils import parse_query_string
 
 from website.common.models import BaseContentPage
-from website.common.utils import TocEntry, get_page_models
+from website.common.utils import get_page_models
 
 from .serializers import MIN_SEARCH_LENGTH, SearchParamsSerializer
 
@@ -29,8 +29,8 @@ class SearchPage(RoutablePageMixin, BaseContentPage):
         return False
 
     @cached_property
-    def table_of_contents(self) -> list[TocEntry]:
-        return []
+    def show_table_of_contents(self) -> bool:
+        return False
 
     def get_context(self, request: HttpRequest) -> dict:
         context = super().get_context(request)

website/spotify/models.py

@@ -41,8 +41,8 @@ class SpotifyPlaylistPage(BaseContentPage):
     ]
 
     @property
-    def table_of_contents(self) -> list:
-        return []
+    def show_table_of_contents(self) -> bool:
+        return False
 
     @cached_property
     def reading_time(self) -> timedelta: