feat: refine standby banner notification styling#297
Merged
Conversation
Condense the standby notification card to a two-line layout: - Drop the boxed bot icon column; a green status dot now carries the standby semantics - Merge status, workspace, and context (task/space/branch/PR) into a single muted meta line that wraps when needed - Replace textual chip kind labels with small icons; full labels move to tooltips and aria-labels - Flatten chips (no pill borders/backgrounds) and keep card texture to a glass blur, inset top highlight, and one soft shadow - Use theme tokens + color-mix instead of hardcoded rgba so light and ember themes render correctly Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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
2 participants
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.
💡 Change Scope
📝 What Does This PR Do?
Refines the standby/banner notification presentation with a few focused visual details:
CHANGELOG.mdafter PR creationThis is a renderer-only UI polish change. It does not change notification state ownership, IPC, persistence, or runtime behavior.
🏗️ Large Change Spec (Required if "Large Change" is checked)
N/A. This is a Small Change scoped to notification component classes and CSS styling.
1. Context & Business Logic
N/A.
2. State Ownership & Invariants
N/A. No state ownership changes.
3. Verification Plan & Regression Layer
Validated with the repository pre-commit gate plus targeted staged checks for the follow-up changelog-only commit.
✅ Delivery & Compliance Checklist
pnpm pre-commitis completely green.DEVELOPMENT.mdarchitectural boundaries.CHANGELOG.mdupdated for feat: refine standby banner notification styling #297.📸 Screenshots / Visual Evidence
Not attached in this automated draft PR. Please upload a screenshot or screen recording in the GitHub PR UI before moving out of draft if visual review evidence is required.
Verification
pnpm line-check:stagedpnpm pre-commit— 249 passed, 47 skipped in E2E; lint, format, typecheck, related tests, build, and pre-commit E2E completed successfully.pnpm line-check:staged,pnpm format-check:staged