docs(tracker-web): tick UX-8 (AppShell) — Core wave UX-2…UX-8 complete (UX-8)

Generated with [Devin](https://cli.devin.ai/docs)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
This commit is contained in:
saravanakumardb1 2026-05-29 07:16:18 -07:00
parent cbd4274a52
commit 81b5ed0baf

View File

@ -208,10 +208,17 @@ pnpm build # final gate
## UX-8 — AppShell (stretch)
- [ ] **8.1** Replace the hand-rolled top nav in `src/app/dashboard/layout.tsx` with `@bytelyst/ui`
- [x] **8.1** Replace the hand-rolled top nav in `src/app/dashboard/layout.tsx` with `@bytelyst/ui`
`AppShell` (`AppShellSidebar`/`AppShellNav`/`AppShellPageHeader`/`AppShellSkipLink` + mobile
toggle), keeping `ProductSwitcher`, user email, sign-out, and adding the ⌘K trigger + theme
toggle. **Verify:** `pnpm typecheck && pnpm lint && pnpm build`; keyboard nav + skip-link work.
(UX-8 `cbd4274a`: tc/lint/test 182 ✓/format/build/e2e 18 ✓. `AppShell` + `AppShellSidebar`/
`AppShellNav`/`AppShellNavItem` (aria-current active route, client nav) / `AppShellMain`
(focusable `#main-content`) / `AppShellSkipLink` + mobile toggle + overlay. Sidebar keeps
`ProductSwitcher`, user email, Sign out and adds a ⌘K trigger (replays the hotkey) + theme
toggle. AppShell exports go through the Primitives adapter (CC.6) + export-presence test.
`AppShellPageHeader` deliberately not used — the per-page `PageHeader` (UX-10) stays the single
h1 per route, avoiding duplicate headings.)
## UX-9 — Complete the Primitives adapter (+ close the coverage gap)
@ -340,7 +347,7 @@ pnpm build # final gate
## Progress
```
Core : UX-1 ✅ UX-2 ✅ UX-3 ✅ UX-4 ✅ UX-5 ✅ UX-6 ✅ UX-7 ✅ UX-8
Core : UX-1 ✅ UX-2 ✅ UX-3 ✅ UX-4 ✅ UX-5 ✅ UX-6 ✅ UX-7 ✅ UX-8
Expand : UX-9 ✅ UX-10 ✅ UX-11 ✅ UX-12 ✅ UX-13 ✅ (stretch: 12.3, 13.1 deferred — see notes)
```