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): room header notify pill + title polish (LC-378) #387

Merged
longjacksonle merged 1 commit from feat/lc-378-room-header-notify into main 2026-06-19 18:28:13 +02:00
Conversation 0 Commits 1 Files changed 2 +12 -5
longjacksonle commented 2026-06-19 18:26:26 +02:00
Owner
Copy link

Closes LC-378. Part of the LC-375 room-view deep-polish epic.

What

The room header's structural alignment landed in LC-373 (shared .lc-header skeleton, lc-h1 title, lc-small topic, .lc-header-action buttons). This finishes the remaining hierarchy items.

  • Notify pill. The old plain bordered word ("Unmuted") becomes a proper header pill: a bell icon, or a bell with a slash when muted, plus the current state label, on the shared .lc-header-action treatment so it matches Info / Moderators in height and hover. Muted state tints warning at rest; the open state tints sunken (via the aria-expanded variant). The bell-slash icon carries the state independent of color.
  • Title. The room-name # becomes a muted prefix span so the name reads as the primary identity.

Preserved

The notify disclosure menu, its radio form, the hx-post to notify-prefs, and the #lc-room-header swap target are all unchanged. No backend or i18n changes (reuses the room-notify-* keys).

Tests

  • ./dev/cargo check, just build-css (confirmed the aria-expanded: variant generates), just test, just test-saas - all green. The room header is not part of the LC-77 message fixtures, so none needed regenerating.

🤖 Generated with Claude Code

Closes LC-378. Part of the LC-375 room-view deep-polish epic. ## What The room header's structural alignment landed in LC-373 (shared `.lc-header` skeleton, `lc-h1` title, `lc-small` topic, `.lc-header-action` buttons). This finishes the remaining hierarchy items. - **Notify pill.** The old plain bordered word ("Unmuted") becomes a proper header pill: a bell icon, or a bell with a slash when muted, plus the current state label, on the shared `.lc-header-action` treatment so it matches Info / Moderators in height and hover. Muted state tints warning at rest; the open state tints sunken (via the `aria-expanded` variant). The bell-slash icon carries the state independent of color. - **Title.** The room-name `#` becomes a muted prefix span so the name reads as the primary identity. ## Preserved The notify disclosure menu, its radio form, the `hx-post` to `notify-prefs`, and the `#lc-room-header` swap target are all unchanged. No backend or i18n changes (reuses the `room-notify-*` keys). ## Tests - `./dev/cargo check`, `just build-css` (confirmed the `aria-expanded:` variant generates), `just test`, `just test-saas` - all green. The room header is not part of the LC-77 message fixtures, so none needed regenerating. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
feat(ui): room header notify pill + title polish (LC-378)
All checks were successful
check-secrets / TruffleHog (push) Successful in 11s
Details
check-secrets / Nosey parker (push) Successful in 11s
Details
check-secrets / Kingfisher (push) Successful in 12s
Details
check-secrets / Kingfisher (pull_request) Successful in 4s
Details
check-secrets / Nosey parker (pull_request) Successful in 4s
Details
check-secrets / TruffleHog (pull_request) Successful in 8s
Details
Create release / Create release from merged PR (pull_request) Has been skipped
Details
Check / clippy + fmt + tests (pull_request) Successful in 7m58s
Details
aebbe5e1fb 
Restyle the room header's notification control from a plain bordered word ("Unmuted") into a proper header pill: a bell icon (or a bell with a slash when muted) plus the current state label, on the shared .lc-header-action treatment so it aligns in height and hover with the Info / Moderators actions. The muted state tints warning at rest and the open state tints sunken (aria-expanded variant); the bell-slash icon carries the state independent of color. The disclosure menu, its radio form, the hx-post to notify-prefs, and the #lc-room-header swap target are all unchanged.

Also tighten the title block: the room-name `#` becomes a muted prefix span so the name itself reads as the primary identity, leaving the lc-h1 name + lc-small topic subline from LC-373 intact.

No backend or i18n changes; reuses the existing room-notify-* keys.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
longjacksonle deleted branch feat/lc-378-room-header-notify 2026-06-19 18:28:13 +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!387
No description provided.