Merge pull request #383 from torchbox/fix/navigation-tags-500

Fix: Check first if the page exists in the context

Author: SharmaineLim

tbx/navigation/templatetags/navigation_tags.py

@@ -34,6 +34,10 @@ def primarynavmobile(context):
 
 @register.simple_tag(name="get_top_level_parent_page", takes_context=True)
 def get_top_level_parent_page(context):
+    # Exit immediately if there is there's no data available.
+    if "page" not in context or "settings" not in context:
+        return None
+
     # Get all page links from the primary nav.
     primary_nav = [
         block.value["page"]

tbx/project_styleguide/templates/patterns/organisms/header/header.html

@@ -20,8 +20,16 @@
                 </nav>
 
                 {# Primary mobile menu toggle #}
-                <button class="header__primary-menu-toggle" data-primary-mobile-menu-toggle aria-haspopup="true" aria-expanded="false" aria-label="Mobile menu toggle">
-                    Home
+                {% get_top_level_parent_page as parent_page %} {# Get the top-level parent page #}
+                {% firstof parent_page.title "Home" as parent_title %} {# Get the parent's title, or Home if no parent #}
+                <button
+                    aria-expanded="false"
+                    aria-haspopup="true"
+                    aria-label="Mobile menu toggle - currently viewing '{{ parent_title }}' or pages below it"
+                    class="header__primary-menu-toggle"
+                    data-primary-mobile-menu-toggle
+                >
+                    {{ parent_title }}
                     {% include "patterns/atoms/icons/icon.html" with name="chevron" classname="header__primary-menu-toggle-icon" %}
                 </button>