Skip to content

fix(desktop): even selection tint and row spacing in the sidebar - #3529

Merged
andreapn merged 1 commit into
mainfrom
fix/sidebar-session-row-spacing
Sep 29, 2026
Merged

andreapn merged 1 commit into
mainfrom
fix/sidebar-session-row-spacing

Conversation

@andreapn

Copy link
Copy Markdown
Contributor

Summary

Two visual issues in the sidebar's multi-selection (Cmd-click / Shift-click):

  • The open chat looked darker than the other selected rows. A selected open chat had its own rule at 32% --primary, against 22% on every other selected row. That rule is gone: every selected row now takes the same 22% tint. The open chat stays identifiable by its filled dot. The selection rule is declared after the open chat's grey, so it wins at equal specificity.
  • Selected rows merged into one block. Rows were stacked with no gap, so adjacent fills touched. .mac-session-item + .mac-session-item now gets margin-top: 2px, so each row reads as its own pill.

Colors are unchanged; only the extra tint step is removed.

Tests

session-row-css.test.ts:

  • new: no data-selected + aria-current rule exists, and the selection rule comes after the open chat's rule
  • new: adjacent rows have a 2px gap
  • kept: across every palette in light and dark, the selection tint stays clearly apart from the open chat's grey (ΔE > 4)

Both new tests fail against main's tokens.css. npm run check in desktop/ passes: typecheck, eslint, prettier, and all 986 vitest tests.

🤖 Generated with Claude Code

In a multi-selection the open chat took a 32% primary tint against 22% on
the other selected rows, so the first row read darker than the rest. Every
selected row now takes the same 22% tint; the open chat keeps its filled
dot. Session rows also sit 2px apart, so adjacent fills read as separate
pills instead of one merged block.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@andreapn
andreapn merged commit 08566ba into main Sep 29, 2026
6 checks passed
@andreapn
andreapn deleted the fix/sidebar-session-row-spacing branch September 29, 2026 15:23
tyler-james-bridges pushed a commit to tyler-james-bridges/agent-os that referenced this pull request Sep 29, 2026
Same-day desktop follow-up to 2026.9.29 carrying the App card's Latest
version row (use-agent-os#3527, via use-agent-os#3528) and the even sidebar selection tint and
row spacing (use-agent-os#3529).

use-agent-os#3528 merged without a CHANGELOG entry; one is added under Added.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant