-
Notifications
You must be signed in to change notification settings - Fork 39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Swapped out hamburger menu for a Login
button in mobile view
#3812
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Like the approach!! The current login page doubles up as signup anyway.
Looks good, happy to merge this. But when testing this I saw things we can improve in the overall login/signup flow/forms. See #3813 |
@frjo I totally agree. Should we merge this as a fix for the time being and hit a refactor next or did you want to handle this now? |
…App#3812) Fixes HyphaApp#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.
…App#3812) Fixes HyphaApp#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.
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.
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.
…App#3812) Fixes HyphaApp#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.
Description
Fixes #3811. This removes the hamburger menu for unauthenticated users in favor of a simple
Login
button. If a signup is intended andENABLE_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.Test Steps
Testing no burger menu when not logged in
Login
button appears in the top rightTesting burger menu for logged in users