psa-systems/lets-chat
Watch
9
0
Fork
You've already forked lets-chat
 0
Code Issues Pull requests Projects Releases 2 Packages Wiki Activity Actions

feat(ui): align + lighten the sidebar room/DM list (LC-381) #393

Merged
longjacksonle merged 1 commit from feat/lc-381-room-list-polish into main 2026-06-20 03:32:37 +02:00
Conversation 0 Commits 1 Files changed 4 +12 -10
longjacksonle commented 2026-06-20 03:32:28 +02:00
Owner
Copy link

Closes LC-381. Final ticket of the LC-375 room-view deep-polish epic.

What

Brings the sidebar room/DM list into one rhythm with the LC-367 navigation rows above it, and lightens the heavy active highlight.

  • Real hover bug fixed. The room/DM rows used hover:bg-surface-sunken, but the sidebar itself is bg-surface-sunken - so the hover was invisible (same color). They now use hover:bg-surface-elevated (+ focus:bg-surface-elevated), matching the nav rows and actually showing a raised hover.
  • One rhythm. Gap/padding move to gap-2.5 / py-1.5 / rounded-md to match the LC-367 nav rows.
  • Lighter active state. The prior full soft-accent (blue) fill read heavy in-room. .lc-room-row-active now uses a neutral raised surface + the accent left bar + an accent-colored bold label - clear but quiet.
  • Tidied add-category control. Input and Add button move onto the shared token treatment (border-border, elevated surface, accent focus ring, a visible button hover instead of the old same-color no-op).

Preserved

Markup/CSS only. aria-current, drag/drop, category move/rename/delete, star, per-room mark-read, and the unread/mention/draft badge wiring are all unchanged. No i18n changes.

Tests

  • ./dev/cargo check, just build-css, just test, just test-saas - all green.

🤖 Generated with Claude Code

Closes LC-381. Final ticket of the LC-375 room-view deep-polish epic. ## What Brings the sidebar room/DM list into one rhythm with the LC-367 navigation rows above it, and lightens the heavy active highlight. - **Real hover bug fixed.** The room/DM rows used `hover:bg-surface-sunken`, but the sidebar itself is `bg-surface-sunken` - so the hover was invisible (same color). They now use `hover:bg-surface-elevated` (+ `focus:bg-surface-elevated`), matching the nav rows and actually showing a raised hover. - **One rhythm.** Gap/padding move to `gap-2.5` / `py-1.5` / `rounded-md` to match the LC-367 nav rows. - **Lighter active state.** The prior full soft-accent (blue) fill read heavy in-room. `.lc-room-row-active` now uses a neutral raised surface + the accent left bar + an accent-colored bold label - clear but quiet. - **Tidied add-category control.** Input and Add button move onto the shared token treatment (`border-border`, elevated surface, accent focus ring, a visible button hover instead of the old same-color no-op). ## Preserved Markup/CSS only. `aria-current`, drag/drop, category move/rename/delete, star, per-room mark-read, and the unread/mention/draft badge wiring are all unchanged. No i18n changes. ## Tests - `./dev/cargo check`, `just build-css`, `just test`, `just test-saas` - all green. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
feat(ui): align + lighten the sidebar room/DM list (LC-381)
All checks were successful
check-secrets / Nosey parker (push) Successful in 3s
Details
check-secrets / Kingfisher (push) Successful in 5s
Details
check-secrets / Kingfisher (pull_request) Successful in 4s
Details
check-secrets / TruffleHog (pull_request) Successful in 6s
Details
check-secrets / TruffleHog (push) Successful in 5s
Details
check-secrets / Nosey parker (pull_request) Successful in 6s
Details
Create release / Create release from merged PR (pull_request) Has been skipped
Details
Check / clippy + fmt + tests (pull_request) Successful in 5m5s
Details
f2503732cf 
Bring the room/DM list into one rhythm with the LC-367 navigation rows above it and lighten the heavy active highlight.

Fixes a real hover bug: the room/DM rows used hover:bg-surface-sunken, but the sidebar itself is bg-surface-sunken, so the hover was invisible (same color). They now use hover:bg-surface-elevated (and focus:bg-surface-elevated), matching the nav rows and actually showing a raised hover. Gap and padding move to gap-2.5 / py-1.5 / rounded-md so the room/DM rows share the nav rows' spacing.

The active room/DM highlight is lightened: the prior full soft-accent (blue) fill read heavy in-room, so .lc-room-row-active now uses a neutral raised surface plus the accent left bar and an accent-colored, bold label - a clear but quiet active state. The add-category input + Add button are tidied onto the shared token treatment (border-border, elevated surface, accent focus ring, a visible button hover instead of the old same-color no-op).

Markup/CSS only; aria-current, drag/drop, category, star, mark-read, and badge wiring are all unchanged. No i18n changes.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
longjacksonle deleted branch feat/lc-381-room-list-polish 2026-06-20 03:32:37 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
psa-systems/lets-chat!393
No description provided.