Skip to content

feat: add app and GitHub links to the navbar, move the docs version to the sidebar - #7

Merged
cavenel merged 1 commit into
mainfrom
feat/navbar
Oct 12, 2026
Merged

cavenel merged 1 commit into
mainfrom
feat/navbar

Conversation

@cavenel

@cavenel cavenel commented Oct 12, 2026

Copy link
Copy Markdown
Contributor

References and relevant issues

None.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Improvement (non-breaking change which modifies an existing feature)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Other

List of changes made

  • An "Open app" button next to the site title opens TissUUmaps from every page, not only from the landing page.
  • A GitHub icon on the right of the navbar links to the TissUUmaps repository. The phone menu shows it with a "GitHub" label.
  • The docs version moved from the navbar, where it sat next to the partner logos, to the top of the docs sidebar. It only applies to the docs.
  • The partner logos show from 1300px instead of 1200px. The new items left too little room at 1200px, so the navbar wrapped with a scrollbar or zoom.
  • The partner logos are now also at the bottom of the phone menu.
  • Site storage is per browser tab (sessionStorage), so a closed TissUUmaps 3 banner comes back in a new tab. The color mode and docs version choices are also per tab.

Screenshot of the fix

Build it locally (see Testing).

Use of AI

Claude Code was used to implement the changes.

Testing

  • A certain branch in other repos is needed: no.
  • Instructions on how to test: pnpm run fmt:check, lint, typecheck and test pass; pnpm run prepare:site --skip-apps && pnpm run build && pnpm run serve. Check the navbar at 1440px, 1100px and 390px wide (phone menu), and a docs page for the version.

Further comments

On phones the logos are pinned to the bottom of the menu with CSS (:has()). If the Docusaurus menu markup changes, they fall back to the end of the list.

@jwindhager jwindhager left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks, lgtm!

@cavenel
cavenel merged commit 6f97821 into main Oct 12, 2026
1 check passed
@jwindhager
jwindhager deleted the feat/navbar branch October 12, 2026 11:40
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.

2 participants