Skip to content

fix(ui): stop account switcher overlapping topbar breadcrumb - #180

Merged
chunter-cb merged 1 commit into
mainfrom
worktree-fix-topbar-account-overlap
Oct 6, 2026
Merged

chunter-cb merged 1 commit into
mainfrom
worktree-fix-topbar-account-overlap

Conversation

@danyalprout

@danyalprout danyalprout commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Ensure the account switcher is never obscured by the header text.

pr180-topbar-desktop.mp4
pr180-topbar-mobile-menu.mp4

@cb-heimdall

cb-heimdall commented Oct 6, 2026 •

Copy link
Copy Markdown
Collaborator

✅ Heimdall Review Status

Requirement Status More Info
Reviews ✅ 1/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@vercel

vercel Bot commented Oct 6, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
omni-ui Ready Ready Preview Oct 6, 2026 4:49pm UTC

Request Review

The switcher is absolutely positioned in the topbar, so on narrower desktop
widths it painted over the centered breadcrumb. Collapse it to avatar +
chevron only when the full label wouldn't fit next to the breadcrumb, and let
the parent crumbs truncate so the current page title stays readable.

On mobile, move the switcher into the hamburger menu instead of the page.
@chunter-cb
chunter-cb added this pull request to the merge queue Oct 6, 2026
Merged via the queue into main with commit 1b09155 Oct 6, 2026
13 checks passed
@chunter-cb
chunter-cb deleted the worktree-fix-topbar-account-overlap branch October 6, 2026 17:11

This branch was successfully deployed

1 active deployment
Preview — 3cd24912 Deployed Oct 6, 2026 by vercel[bot]
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.

3 participants