Merge pull request #604 from GSA/602-no-top-nav-for-logged-in-users

bug: Adjust conditional code to show nav when user is logged in
This commit is contained in:
Carlo Costino
2023-07-07 14:36:38 -04:00
committed by GitHub
3 changed files with 6 additions and 5 deletions

View File

@@ -110,7 +110,8 @@
] %}
{% endif %}
{% else %}
{% set navigation = [
<!-- Add navigation back after pilot -->
{# {% set navigation = [
{
"href": url_for('main.get_started'),
"text": "Using notify",
@@ -131,7 +132,7 @@
"text": "Sign in",
"active": header_navigation.is_selected('sign-in')
}
] %}
] %} #}
{% endif %}
{{ govukHeader({

View File

@@ -1,4 +1,4 @@
<fieldset class="govuk-fieldset margin-bottom-5
<fieldset class="govuk-fieldset margin-y-5
{%- if params.classes %} {{ params.classes }}{% endif %}"
{%- if params.describedBy %} aria-describedby="{{ params.describedBy }}"{% endif %}
{%- for attribute, value in params.attributes %} {{ attribute }}="{{ value }}"{% endfor %}>

View File

@@ -71,7 +71,7 @@
</div>
<button type="button" class="usa-menu-btn">Menu</button>
</div>
{# <nav aria-label="Primary navigation" class="usa-nav">
<nav aria-label="Primary navigation" class="usa-nav">
<div class="usa-nav__inner">
<button type="button" class="usa-nav__close">
<img src="/static/images/usa-icons/close.svg" role="img" alt="Close" />
@@ -117,5 +117,5 @@
</section> -->
</div>
</div>
</nav> #}
</nav>
</header>