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

Allow top_links to wrap if too many for space #612

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

will-moore
Copy link
Member

Fixes #611.

If we have too many top-links, the header wraps to another line.

NB: the width of the top-links list is slightly hard-coded to be 550px less than the width of the page.
Probably best to use flex layout for the header row to be more flexible (e.g. size of logo and login etc).

Screenshot 2025-02-28 at 17 03 42

@joshmoore
Copy link
Member

How hard would it be for this to become a drop-down menu?

@will-moore
Copy link
Member Author

@joshmoore It's doable. But under what conditions?

  • All links are always shown in a drop-down menu?
  • Links above a certain count go into a More links... dropdown menu?
  • Or, it depends on the available space (screen size)? - bit more work.

NB: adding "flex" layout to body causes layout to change on login page:

Screenshot 2025-03-24 at 13 06 59

@joshmoore
Copy link
Member

I'd personally prefer any of the options you mentioned over the wrapping behavior.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make the middle header bar thicker to add more elements on it
2 participants