Fix for secondary navigation not showing correctly (#120)

* Fix for secondary navigation not showing correctly
* Swap for a comment about why we can't conditionally render this

Author: edwardhorsford

app/views/_templates/layout-app.html

@@ -149,14 +149,17 @@
 </div>
 {% endif %}
 
-{% if pageNavigation %}
-  <div class="nhsuk-grid-row">
-    <div class="nhsuk-grid-column-full">
-      {% block pageNavigation %}
-      {% endblock pageNavigation %}
-    </div>
+
+{# This will output excess divs where pageNavigation is not set. I could not find a way to stop this. You can't test for truthiness of the block, nor can you capture the contents with set.
+
+See related issues:
+https://github.com/mozilla/nunjucks/issues/135
+https://github.com/mozilla/nunjucks/issues/164#issuecomment-3073581360 #}
+<div class="nhsuk-grid-row">
+  <div class="nhsuk-grid-column-full">
+    {% block pageNavigation %}{% endblock pageNavigation %}
   </div>
-{% endif %}
+</div>
 
 {% if gridColumn != "none" %}
 <div class="nhsuk-grid-row">

app/views/_templates/layout-reading.html

@@ -174,7 +174,6 @@
     <div class="nhsuk-u-margin-bottom-4"></div>
     {% include "reading/secondary-navigation.njk" %}
 
-
   {% endif %}
 {% endblock pageNavigation %}