Agentic UI: Make sidebar notices legible and add a recent notifications dialog - #4777
Open
shaunandrews wants to merge 1 commit into
Open
Agentic UI: Make sidebar notices legible and add a recent notifications dialog#4777shaunandrews wants to merge 1 commit into
shaunandrews wants to merge 1 commit into
Conversation
wojtekn
marked this pull request as ready for review
September 9, 2026 07:37
4 tasks
Contributor
|
@shaunandrews see #4808 with proposed changes. |
wojtekn
self-requested a review
September 9, 2026 08:39
wojtekn
added this pull request to stack #4809
September 9, 2026 09:56
wojtekn
force-pushed
the
claude/sidebar-notice-visibility-37d2be
branch
from
September 9, 2026 12:10
804cfe7 to
3228725
Compare
Collaborator
📊 Performance Test ResultsComparing 2fb5f9c vs trunk app-size
site-editor
site-startup
Results are median values from multiple test runs. Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change (<50ms diff) |
…ns dialog Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
wojtekn
force-pushed
the
claude/sidebar-notice-visibility-37d2be
branch
from
September 9, 2026 12:50
3228725 to
2fb5f9c
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Related issues
How AI was used in this PR
Built with Claude Code, iterating in a running
studio uiagainst a temporary tweaks panel that simulated notices of every intent and length (including a 300-line stack trace). That panel is not part of this PR. Every visual state below was reviewed in light and dark; the diff, copy, and tests were reviewed by hand.Proposed Changes
A few problems this tries to solve:
Solutions:
Long errors no longer swallow the UI. A toast description now clamps to three lines. Error toasts that carry details gain Copy (title plus the full text, for pasting into a bug report) and More, which opens a new Recent notifications dialog from the bell beside App settings.
The dialog is a session log, not a persisted inbox: everything Studio has shown you, newest first, with the full text of each notice, a timestamp, and a per-entry Copy. A single entry scrolls internally past roughly a dozen lines, so one enormous stack trace cannot push the rest off screen. Clear all empties the list and dismisses the toasts still showing or queued, so the sidebar agrees with the dialog. With nothing to show it presents an empty state rather than an empty list, and Clear all is not offered.
One 300-line stack trace scrolls inside its own entry rather than pushing the rest of the list away, and the empty state stands in for an empty list.
Testing Instructions
Run the agentic UI (
npm run cli:build:ui && node apps/cli/dist/cli/main.mjs ui --no-open) or the Desktop app, in both light and dark.Pre-merge Checklist