heaven/vojo
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

fix(workspace-switcher): switch space rows to SurfaceVariant so inactive bg blends with the sheet silhouette instead of reading as dark cards

Browse source
This commit is contained in:
heaven 2026-05-21 01:05:34 +03:00
parent 78504262d3
commit 055b7d3692
1 changed files with 8 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
src/app/pages/client/channels/WorkspaceSwitcherSheet.tsx
View file

@ -70,11 +70,13 @@ type SpaceRowProps = {
};
// One space row in the sheet. Mirrors the DM-row visual: avatar 40,
// two text lines (name + member-count subtitle). The active community
// is signalled solely via `aria-selected` — Folds NavItem paints
// `ContainerActive` on `&[aria-selected=true]`, which is enough on
// this surface tier; an explicit trailing label was tried and
// dropped per product call.
// two text lines (name + member-count subtitle). Variant is
// `SurfaceVariant` so the inactive row bg (`SurfaceVariant.Container`
// = #181a20) matches the sheet silhouette and blends into it — same
// treatment as `CreateCommunityRow` above. The active community is
// signalled solely via `aria-selected` → `ContainerActive` (#2a2d36),
// which lifts cleanly off the sheet without a card outline; an
// explicit trailing label was tried and dropped per product call.
//
// `useRoomName` lives here so the per-space subscription stays scoped
// (Rules of Hooks) and admin renames reflect in the dropdown without
@ -86,7 +88,7 @@ function SpaceRow({ space, isActive, onPick }: SpaceRowProps) {
return (
<NavItem
variant="Background"
variant="SurfaceVariant"
radii="400"
aria-selected={isActive}
style={{ minHeight: ROW_MIN_HEIGHT }}