Skip to content
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

[20.09] Fix masthead items disappearing on small screens #10641

Merged

Conversation

mvdbeek
Copy link
Member

@mvdbeek mvdbeek commented Nov 6, 2020

The fix is to not set toggleable on <b-navbar>. I also played with with adding a <b-navbar-toggle/> to display a button on the right for small screens, but I didn't manage to get something that looks nice, though it's probably not hard for someone that knows css.

To get a little bit more space in the navbar I'm setting the (non-functional) chat server item to display: none and I'm moving the scratchbook to the rightmost position (since it reserves some space just on the right to display active scratchbooks).

With this in place I see the full navbar on my phone.

Fixes #9844.

Since it reserves some space to show active scratchbooks. This removes
the weird space in between the scratchbook and the masthead extensions.
@mvdbeek mvdbeek changed the title [20.09] Fix masthead disappearing on small screens [20.09] Fix masthead items disappearing on small screens Nov 6, 2020
@nsoranzo nsoranzo added this to the 20.09 milestone Nov 6, 2020
Copy link
Member

@dannon dannon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is fine for a 20.09 fix, but we should follow up and fix togglable and use the hamburger properly in dev.

@dannon dannon merged commit c49d4f7 into galaxyproject:release_20.09 Nov 9, 2020
@martenson martenson deleted the fix_masthead_disappearing branch November 11, 2020 18:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants