Skip to content

fix(dashboard): Keep the header the same width on every page - #2151

Open
sentry-junior[bot] wants to merge 2 commits into
mainfrom
fix/dashboard-header-stable-width
Open

sentry-junior[bot] wants to merge 2 commits into
mainfrom
fix/dashboard-header-stable-width

Conversation

@sentry-junior

@sentry-junior sentry-junior Bot commented Oct 15, 2026

Copy link
Copy Markdown
Contributor

The primary nav jumped when you moved between Conversations and other pages. The Conversations workspace rendered the header full width (compact), while Code, Automations, Memories, and System put it in the centered max-w-screen-xl container with different padding. The logo, nav links, and spend/profile control all moved on each route change.

The header now uses one full-width frame with the same padding on every route, so the nav stays in place. I removed the compact prop because nothing else needs it. Page content keeps its current centered width.

via richard.

--

View Junior Session [Sentry]

Co-Authored-By: Richard Roggenkemper <richard.roggenkemper@sentry.io>
@vercel

vercel Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
junior-docs Ready Ready Preview Oct 15, 2026 5:05pm UTC

Request Review

@sentry-junior sentry-junior Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This fixes the jump between Conversations and Code, but the secondary navigation strip is now out of line with the header.

The desktop strip in SecondaryNavigation (System, Automations, Memories, Code repository pages) still uses dashboardContainerClass with md:px-8. Before this PR it used the same frame as the header, so the two rows lined up. Now the primary nav sits at the left edge and the sub-tabs (Overview / People / Locations / …) start about 230px to the right of it at 1728px wide. Users will see a new misalignment on each page that has sub-tabs.

Fix: give the desktop strip in SecondaryNavigation.tsx the same full-width frame as the header (w-full min-w-0 px-4 py-2 md:px-7). Then the sub-tabs line up under the primary nav, and page content keeps its centered width. The rest of the change looks right. Removing compact is clean, and the mobile classes did not change.

@github-actions

github-actions Bot commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

54 screenshot changes — 54 changed · 0 added · 0 removed

Review screenshots in Frameshift

Automation Attention · Desktop
Automation Attention · Desktop
Changed
Automation Editor · Desktop
Automation Editor · Desktop
Changed
Automation Editor Conflict · Desktop
Automation Editor Conflict · Desktop
Changed
Automation Editor Conflict · Mobile
Automation Editor Conflict · Mobile
Changed
Automation Editor Public · Desktop
Automation Editor Public · Desktop
Changed
Automation Event Editor · Desktop
Automation Event Editor · Desktop
Changed
Show 12 more changed screenshots
Automation Executions · Desktop
Automation Executions · Desktop
Changed
Automation Paused · Desktop
Automation Paused · Desktop
Changed
Automation Versions · Desktop
Automation Versions · Desktop
Changed
Automations · Desktop
Automations · Desktop
Changed
Automations List · Desktop
Automations List · Desktop
Changed
Automations Runs · Desktop
Automations Runs · Desktop
Changed
Code · Desktop
Code · Desktop
Changed
Code Repository · Desktop
Code Repository · Desktop
Changed
Conversation Attachment · Desktop
Conversation Attachment · Desktop
Changed
Conversation Attachment Modal · Desktop
Conversation Attachment Modal · Desktop
Changed
Conversation Child Agent Drawer · Desktop
Conversation Child Agent Drawer · Desktop
Changed
Conversation Child Agent Page · Desktop
Conversation Child Agent Page · Desktop
Changed

36 more changed screenshots in Frameshift.

Co-Authored-By: Richard Roggenkemper <richard.roggenkemper@sentry.io>

This branch was successfully deployed

1 active deployment
Preview – junior-docs — 30f03fa1 Deployed Oct 15, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk: medium PR risk score: medium

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants