Agentic UI: flatten notices to one neutral card, retitle the history dialog - #4808
Open
wojtekn wants to merge 3 commits into
Open
Agentic UI: flatten notices to one neutral card, retitle the history dialog#4808wojtekn wants to merge 3 commits into
wojtekn wants to merge 3 commits into
Conversation
1 task
wojtekn
added this pull request to stack #4809
September 9, 2026 09:56
wojtekn
force-pushed
the
flatten-sidebar-notice-colors
branch
from
September 9, 2026 12:10
50ee236 to
d28a7d0
Compare
wojtekn
force-pushed
the
flatten-sidebar-notice-colors
branch
from
September 9, 2026 12:50
d28a7d0 to
eaf94ce
Compare
The intent icon already carries the severity, so the tinted surface adds color noise without adding information. Notices keep a single quiet grey card in the sidebar and the notifications dialog. The sidebar's card is lifted off the chrome rather than off the app theme: the chrome is dark in both color schemes, so the notice stays the same grey in light and dark instead of flipping to near-white. The dialog sits on the app canvas, so its own lift follows the color scheme as usual. This also drops the now-unused 'appearance' prop from AppToasts and AppMessageCards.
'Recent notifications' plus 'Notices from this session, newest first' said the same thing twice: a history is newest-first by definition, and the list itself shows the ordering. The title now names the thing, and the subtitle keeps only the fact the title can't carry — that the log is session-scoped and does not survive a restart.
As a subtitle it sat between the title and the notices, in the way of the thing the reader opened the dialog for. It is a one-time fact about the log's lifetime, so it reads better as a quiet footnote above the buttons — smaller and dimmer than the notice text it follows.
wojtekn
force-pushed
the
flatten-sidebar-notice-colors
branch
from
September 10, 2026 10:42
eaf94ce to
5af83f2
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
1 participant
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 against a running
studio ui, using a temporarywindow.__toastshook to fire notices of each intent (not part of this PR). Colors below were read back withgetComputedStyle, not eyeballed.Proposed Changes
Flatten notices to one neutral card
I propose using a neutral card color and using an icon to mark severity. Also, as the sidebar has the same color in light and dark mode, I propose using the same color for notifications.
The dialog follows the light/dark mode scheme, so colors are adjusted there.
Retitle the dialog, demote the retention note
"Recent notifications" + "Notices from this session, newest first…" said the same thing twice. I retitled it to Notification history (bell tooltip follows).
The session-scoped fact is real but secondary, and as a subtitle it sat between the title and the notices. I move below the list as a footnote above the buttons: "Notifications are cleared automatically when Studio restarts."
Testing Instructions
Run the agentic UI (
npm start), in both light and dark.Pre-merge Checklist
npm run typecheckpassesnotice-history,app-toasts,app-messages🤖 Generated with Claude Code