fix(dashboard): Keep mobile conversations at latest - #1549
fix(dashboard): Keep mobile conversations at latest#1549sentry-junior[bot] wants to merge 4 commits into
Conversation
Co-Authored-By: Pierre Massat <pierre.massat@sentry.io>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Dashboard visual evidenceMode: path-selected Triggered by:
Conversations · desktop
Conversations · mobile
Conversation detail · desktop
Conversation detail · mobile
Component gallery · desktop
Full-page screenshots from the mock dashboard. Not a pixel-diff gate. |
Pin the transcript when the pending stack resizes, keep the composer outside the queue scroller, and anchor the reply footer to the bottom of the mobile conversation shell. Co-Authored-By: Pierre Massat <pierre.massat@sentry.io>
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
![sentry-warden[bot]](https://avatars.githubusercontent.com/in/2755838?s=60&v=4){kind=small}
…on-bottom-follow # Conflicts: # packages/junior-dashboard/src/client/conversations/ConversationPage.tsx # packages/junior-dashboard/src/client/conversations/PendingMailboxStack.tsx Co-Authored-By: Pierre Massat <pierre.massat@sentry.io>
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
There are 2 total unresolved issues (including 1 from previous review).
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 665e194. Configure here.
| <div | ||
| aria-label="Conversation transcript" | ||
| className="min-h-0 overflow-y-auto overscroll-contain px-3 pb-3 md:px-7 md:pb-5" | ||
| className="min-h-0 flex-1 overflow-y-auto overscroll-contain px-3 pb-3 md:px-7 md:pb-5" |
There was a problem hiding this comment.
Transcript can collapse on desktop
Low Severity
Replacing the grid minmax(7rem, 1fr) transcript row with a flex-1 min-h-0 sibling lets the reply footer consume the whole shell when the pending queue is tall. Desktop removes the footer max-h, so an expanded mailbox can shrink the transcript to zero.
Reviewed by Cursor Bugbot for commit 665e194. Configure here.
There was a problem hiding this comment.
Fixed in bec29a9.
Restored the page shell to grid-rows-[minmax(7rem,1fr)_minmax(0,auto)], so the transcript keeps a minimum row and a tall queue cannot collapse it on desktop. The sticky composer/footer layout from main remains.
Gate mobile always-follow on live mode, and restore the grid shell so a tall queue cannot collapse the transcript on desktop. Co-Authored-By: PDPM <pierre.massat@sentry.io>
Keep mobile conversations pinned to the latest message, and keep the composer at the bottom above the pending queue.
What changed
Desktop follow behavior stays unchanged.
Verified
Requested by PDPM.
--
View Junior Session [Sentry]