Skip to content

feat(dashboard): Stack sidebar annotations - #1552

Merged
dcramer merged 12 commits into
mainfrom
feat/stack-sidebar-annotations
Aug 14, 2026
Merged

feat(dashboard): Stack sidebar annotations#1552
dcramer merged 12 commits into
mainfrom
feat/stack-sidebar-annotations

Conversation

@sentry-junior

@sentry-junior sentry-junior Bot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

GitHub no longer collapses conversation-row annotations into one summary chip.

The plugin returns every annotation newest-first. The dashboard stacks them left to right with the newest fully visible and older chips partially covered.

Stacked chips use compact repository names so more of the stack can stay readable. Desktop hover shows the full resource keys. Mobile stays tooltip-free.

Mock fixtures now include multi-annotation stacks for local visual QA.

Requested by David Cramer.

--

View Junior Session [Sentry]

@vercel

vercel Bot commented Aug 14, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
junior-docs Ready Ready Preview Aug 14, 2026 3:40am

Request Review

@dcramer
dcramer marked this pull request as ready for review August 14, 2026 01:30
@github-actions github-actions Bot added the risk: medium PR risk score: medium label Aug 14, 2026
@github-actions

github-actions Bot commented Aug 14, 2026

Copy link
Copy Markdown

Dashboard visual evidence

Mode: path-selected
Selected: conversations, conversation-detail

Triggered by:

  • packages/junior-dashboard/e2e/conversations.spec.ts
  • packages/junior-dashboard/src/client/conversations/ConversationMeta.tsx
  • packages/junior-dashboard/src/client/conversations/ConversationPage.tsx
  • packages/junior-dashboard/src/client/conversations/ConversationSidebar.tsx
  • packages/junior-dashboard/src/client/conversations/PendingMailboxStack.tsx
  • packages/junior-dashboard/src/mock-reporting/fixtures.ts
  • packages/junior-dashboard/tests/dashboard-mock-routes.test.ts
  • packages/junior-dashboard/tests/pending-mailbox-stack.test.tsx

Conversations · desktop

Conversations · desktop

Conversations · mobile

Conversations · mobile

Conversation detail · desktop

Conversation detail · desktop

Conversation detail · mobile

Conversation detail · mobile

Full-page screenshots from the mock dashboard. Not a pixel-diff gate.

@blacksmith-sh

This comment has been minimized.

sentry-junior Bot and others added 8 commits August 14, 2026 03:02
Co-Authored-By: David Cramer <david@sentry.io>

Co-Authored-By: David Cramer <david@sentry.io>
Show compact repository names on stacked sidebar chips and keep the full resource key in the desktop tooltip.

Co-Authored-By: David Cramer <david@sentry.io>

Co-Authored-By: David Cramer <david@sentry.io>
Keep sidebar linked-work chips as a compact icon stack and move labels into the desktop tooltip. Hide channel location when annotations already own the meta row.

Co-Authored-By: David Cramer <david@sentry.io>

Co-Authored-By: David Cramer <david@sentry.io>
The stacked annotation tooltip can stay open on the selected row, so the e2e cost-metric assertion must match the cost tooltip specifically.

Co-Authored-By: David Cramer <david@sentry.io>
Keep single and dual-repo stacks readable with repo names, collapse same-repo chips, and only fall back to icon peeks for longer stacks. Tooltip still lists every annotation.

Co-Authored-By: David Cramer <david@sentry.io>

Co-Authored-By: David Cramer <david@sentry.io>
On desktop, keep one or two repo labels and move extra scopes into a separate +N icon cluster. On mobile, render a compact status facepile only. Tooltip still lists every annotation.

Co-Authored-By: David Cramer <david@sentry.io>

Co-Authored-By: David Cramer <david@sentry.io>
Use opaque status-color discs with a 2px box-shadow cutout and fixed
16px geometry so stacked annotations read as a clean avatar facepile
instead of nested stroke icons fighting each other.

Co-Authored-By: David Cramer <david@sentry.io>

Co-Authored-By: David Cramer <david@sentry.io>
Keep the solid disc geometry and cutout ring, but put the colored
status glyphs back on each disc so the stack matches labeled chips.

Co-Authored-By: David Cramer <david@sentry.io>

Co-Authored-By: David Cramer <david@sentry.io>
Dashboard style check rejects absolute arbitrary font sizes below the
named scale, so the no-icon disc letter uses text-2xs.
Match labeled-chip border/fill on icon-only stack members and increase
overlap so the overflow cluster reads as nested chips, not floating
glyphs.

Co-Authored-By: David Cramer <david@sentry.io>
Co-Authored-By: David Cramer <david@sentry.io>
Co-Authored-By: David Cramer <david@sentry.io>

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 4849445. Configure here.

@dcramer
dcramer merged commit 0eb3a3f into main Aug 14, 2026
39 checks passed
@dcramer
dcramer deleted the feat/stack-sidebar-annotations branch August 14, 2026 03:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk: high PR risk score: high

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant