backdrop / backdrop-issues

Issue tracker for Backdrop core.
145 stars 40 forks source link

[UX] Basis: login page tabs wrapping outside their container on mobile portrait mode. #3108

Open klonos opened 6 years ago

klonos commented 6 years ago

Mobile landscape:

screencapture-1831-backdrop-backdrop-qa-backdropcms-org-user-login-2018-05-14-10_11_52

Mobile portrait ("Log in" tab):

screencapture-1831-backdrop-backdrop-qa-backdropcms-org-user-login-2018-05-14-10_16_25

Mobile portrait ("Reset password" tab):

image

We (mostly @wesruv actually 😄) did a really great job with the mobile version of the primary tabs in our admin theme in #671. Perhaps we could reuse this here too?

...hiding some of the tabs might make sense to people used to this feature from the admin backend, but it might not be apparent to site visitors. We could consider exposing the "Reset password" as a link next to the "Log in" button(?).

Thoughts?

jenlampton commented 6 years ago

I'd really like to re-work the log-in page to remove the tabs entirely. I almost always remove them in my custom themes these days, and replace them with a markup element containing links to those pages.

See https://github.com/backdrop/backdrop-issues/issues/3309, and maybe that can be the solution to this bug :)

kreynen commented 6 years ago

Are there PRs to review? We took over maintaining https://www.drupal.org/project/noreqnewpass because with SAML or LDAP, also using local accounts creates an unmaintainable mess. This is true for the GitHub SSO as well. Please keep the SSO use case in mind while making these changes.

It is much easier to alter away a menu in a module than it is to figure out if we need to remove markup from a theme.

jenlampton commented 6 years ago

Are there PRs to review?

No PRs yet for https://github.com/backdrop/backdrop-issues/issues/3309