fix(sidebar): keep selected agent row visually distinct from its project card#685
Merged
Merged
Conversation
…tion In the WITH AGENTS card, the always-active project header and the selected agent row both used the same --sidebar-active-bg fill and a 2px accent left bar at the same x-position, so the two highlights read as one merged surface. When any non-header row in the card is active, the header now drops its fill and bar — the elevated card still marks the active project, and the filled row marks the selection. Hover feedback on the header is preserved. Closes #682 Co-Authored-By: Claude Opus 4.8 <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
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.
Summary
In the sidebar's WITH AGENTS section, the always-active project header and the selected agent row both used the same
--sidebar-active-bgfill plus a 2px accent left bar at the same x-position, separated by ~2px — the two highlights read as one merged surface (see #682).Two CSS rules in
theme.css: when any non-header row inside a--has-agentscard is active, the header drops its fill and accent bar (border-left-color: transparentkeeps layout stable). The elevated card still marks the active project; the filled row marks the selection. A companion:hoverrule preserves hover feedback on the quieted header, which the higher-specificity transparent rule would otherwise suppress.The selector also covers workspace cards (
WorkspaceList) and draft rows, which share the same classes. The header keeps its--activeclass, so its action buttons and the behind-origin fetch badge stay visible.Testing
typecheck:webcleanCloses #682
🤖 Generated with Claude Code