Naiyuan Qing f7fe0829f2 refactor(skills): wrap list as card, use shared PageHeader, add scroll fade (#1614) ...

The skills page rolled its own HeroHeader instead of the shared PageHeader,
which meant no mobile sidebar trigger and visual drift from other list
pages. The table was also edge-to-edge inside the dashboard container, so
it felt "naked" compared to the rest of the product.

- Replace custom HeroHeader with shared PageHeader (gives mobile hamburger
  and h-12 chrome for free); move "New skill" into the PageHeader as the
  page-level action.
- Keep search + scope filters in a toolbar, but move that toolbar *inside*
  a bordered, rounded card together with the table, so the whole unit
  reads as a single scrollable surface with internal padding.
- Use the existing useScrollFade hook on the row list so the top/bottom
  edges fade while scrolling.
- Drop `divide-y` in favor of `border-b` per row — divide-y leaves the
  last row without a bottom rule, which looks unfinished when only a
  couple of skills exist and the scroll area is taller than the content.
- Drop the redundant description paragraph from the old hero; keep the
  "Shared with your workspace" banner above the card since it carries
  non-obvious UX context.

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

2026-04-24 15:04:09 +08:00

..

core

refactor(skills): redesign list page and add skill detail page (#1607)

2026-04-24 13:51:58 +08:00

eslint-config

chore: add shared ESLint config + enforce strict tsconfig across packages

2026-04-10 08:27:29 +08:00

tsconfig

chore: add shared ESLint config + enforce strict tsconfig across packages

2026-04-10 08:27:29 +08:00

ui

fix(ui): stop menu hover from overriding icon colors (#1612)

2026-04-24 14:26:58 +08:00

views

refactor(skills): wrap list as card, use shared PageHeader, add scroll fade (#1614)

2026-04-24 15:04:09 +08:00