Skip to content

fix(dashboard): Anchor the Inbox count badge to the icon - #2609

Merged
thomasjiangcy merged 1 commit into
mainfrom
fix/navrail-inbox-badge-icon
Nov 3, 2026
Merged

thomasjiangcy merged 1 commit into
mainfrom
fix/navrail-inbox-badge-icon

Conversation

@sentry-junior

@sentry-junior sentry-junior Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

Follow-up to #2586. The nav rail count badge sat on the top-right corner of the 40px button, so it read as part of the button, not the Inbox icon.

The badge is now at top-0 right-0 of the 40px item. Its center is 2px outside the top-right corner of the 20px icon, so it overlaps the icon corner and most of the glyph stays visible. 99+ still grows to the left over the icon.

Checked in a local Playwright run of the dashboard harness with a mocked review count, in the default and active (/review) states.

via @thomasjiangcy.

--

View Session [Sentry]

The nav rail badge sat on the top-right corner of the 40px button,
so it floated away from the Inbox icon. Center it near the top-right
corner of the 20px icon instead.

Co-Authored-By: Thomas Jiang <20501235+thomasjiangcy@users.noreply.github.com>
@vercel

vercel Bot commented Nov 3, 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 Nov 3, 2026 2:32pm UTC

Request Review

@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

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

Review screenshots in Frameshift

Conversation Attachment · Desktop
Conversation Attachment · Desktop
Changed
Conversation Attachment · Mobile
Conversation Attachment · Mobile
Changed
Conversation Attachment Modal · Desktop
Conversation Attachment Modal · Desktop
Changed
Conversation Sidebar Review Chip · Desktop
Conversation Sidebar Review Chip · Desktop
Changed
Conversation Sidebar Review Chip Light · Desktop
Conversation Sidebar Review Chip Light · Desktop
Changed
Conversation Unsent Answers · Desktop
Conversation Unsent Answers · Desktop
Changed
Show 7 more changed screenshots
Conversations Sections Collapsed · Desktop
Conversations Sections Collapsed · Desktop
Changed
Conversations Waiting First · Desktop
Conversations Waiting First · Desktop
Changed
Review Desk · Desktop
Review Desk · Desktop
Changed
Review Desk Empty · Desktop
Review Desk Empty · Desktop
Changed
Review Desk Light · Desktop
Review Desk Light · Desktop
Changed
Review Desk Search · Desktop
Review Desk Search · Desktop
Changed
Settings Mcp · Desktop
Settings Mcp · Desktop
Changed

@thomasjiangcy
thomasjiangcy merged commit 93fc75c into main Nov 3, 2026
45 checks passed
@thomasjiangcy
thomasjiangcy deleted the fix/navrail-inbox-badge-icon branch November 3, 2026 14:37

This branch was successfully deployed

1 active deployment
Preview – junior-docs — ad3de1c5 Deployed Nov 3, 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.

1 participant