# SidebarNav

## 2026-06-22 — Collapsible group navigates + shows active

**Prompted by:** Hellen (design team)

### What changed
- Clicking a collapsible group header (e.g. "Kanban Boards") now also calls
  `onNavigate(item.href)` — previously it only expanded the accordion, so the
  group's own landing page never opened.
- The group highlights as active when its **own** page is active (`item.isActive`),
  not only when a sub-item is active. New `isGroupActive = hasActiveChild || item.isActive`
  drives the row/icon highlight and auto-expand (rail + expanded modes).

### Why
- Selecting the parent entry should open its landing (the Board Grid) and show
  an active state, not just toggle the sub-list.

### Affected tokens / files
- `packages/shadcn/src/components/ui/sidebar-nav.tsx`

## 2026-06-22 — Readable sub-item hover

**Prompted by:** Hellen (design team)

### What changed
- Sub-item (accordion child) hover text now uses `accent-foreground` instead of
  `brand-secondary-foreground`. The ghost button's hover background is the light
  `accent`, so the light brand-secondary text was unreadable (light-on-light).
- The sub-item's ChevronRight darkens on hover too (`group-hover:text-accent-foreground`).

### Why
- On hover the sub-item background is light; per the theme rule a light
  (`accent`) background must pair with the dark `accent-foreground` text/icon.

### Affected tokens / files
- `packages/shadcn/src/components/ui/sidebar-nav.tsx`