Skip to content
Snippets Groups Projects
Unverified Commit db643d19 authored by Wes Appler's avatar Wes Appler Committed by GitHub
Browse files

Swapped out hamburger menu for a `Login` button in mobile view (#3812)

Fixes #3811. This removes the hamburger menu for unauthenticated users
in favor of a simple `Login` button. If a signup is intended and
`ENABLE_PUBLIC_SIGNUP` is enabled on the instance, the signup button can
be found on the `/auth/` view. I didn't want to overload the mobile view
by putting login & signup next to eachother.
parent 3f10a7be
No related branches found
No related tags found
1 merge request!76Merge up to 5.6.0
......@@ -42,10 +42,19 @@
</a>
<div class="header__inner header__inner--mobile-buttons">
<button class="button button--left-space js-mobile-menu-toggle" aria-haspopup="true">
<span class="sr-only">{% trans "Open Menu" %}</span>
{% heroicon_outline "bars-3" size=32 stroke_width=2 class="inline" aria_hidden="true" %}
</button>
{% if request.user.is_authenticated %}
<button class="button button--left-space js-mobile-menu-toggle" aria-haspopup="true">
<span class="sr-only">{% trans "Open Menu" %}</span>
{% heroicon_outline "bars-3" size=32 stroke_width=2 class="inline" aria_hidden="true" %}
</button>
{% else %}
{% if request.path != '/auth/' %}
{% include "includes/login_button.html" %}
{% endif %}
{% if ENABLE_PUBLIC_SIGNUP and request.path == '/auth/' %}
{% include "includes/register_button.html" %}
{% endif %}
{% endif %}
</div>
{% block header_menu %}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment