Skip to content

redesign(ui): extensions page joins the premium checklist (CSS wave) - #2012

Merged
jackwener merged 1 commit into
mainfrom
redesign/skills-premium
Aug 3, 2026
Merged

redesign(ui): extensions page joins the premium checklist (CSS wave)#2012
jackwener merged 1 commit into
mainfrom
redesign/skills-premium

Conversation

@jackwener

Copy link
Copy Markdown
Member

Wave 2 of the premium-quality goal — the extensions page's pure-CSS checklist violations:

  • Rhythm: flat 8px page stack → two-tier rhythm (24px between banner/tabs/inspector/market/installed tiers).
  • Hover lift retired (rule 17): market cards stop translateY-ing and growing a hand-written shadow on hover; border tone carries the hover. The orphaned --lift-hover token is removed with a tombstone.
  • Hand-written icon-tile shadows removed (market icon + library status tile) — the hairline is the edge.
  • Caption roles corrected (rule 14): five heading-5-as-caption sites → supporting/label roles.

Deliberately untouched: the featured-banner sticker collage (documented product art, z-index contract-tested) and the skills-panel TSX (Token chips, hand-rolled list → row primitives) — queued as the follow-up wave.

Diff self-reviewed per the standing review instruction. Verification: build ✅ typecheck ✅ format:check ✅ dead-css ✅ smoke 74 renders × 3 viewports ✅. Screenshot in the work thread.

Second execution wave of the premium-quality goal, scoped to the
checklist violations that live purely in CSS:

- Page rhythm: the flat 8px stack (banner / tabs / inspector / market /
  installed all identical) becomes the two-tier rhythm — 24px between
  page tiers, small steps inside units; the installed section label
  gets air above its list.
- Hover lift retired (checklist rule 17): market cards no longer
  translateY + grow a hand-written shadow on hover — border tone is
  the whole hover story. The now-orphaned --lift-hover token is
  removed with a tombstone (its own doc called snap lifts out).
- Hand-written icon-tile shadows removed (market icon + library status
  tile) — the hairline border is the edge.
- Caption roles corrected (rule 14): five heading-5-as-caption sites
  (market slug/meta/foot, section-row count, context-inspector title,
  installed section label) move to the supporting/label roles.

Deliberately untouched this wave: the featured-banner sticker collage
(documented product art, z-index contract-tested) and the skills-panel
TSX (Token chips, list conversion) — queued as the follow-up wave.

Self-reviewed the diff per the standing review instruction: e2e
selectors (.maka-skill-library-list) and the z-index allowlist are
unaffected; dead-css/typecheck/format/smoke all green.
@jackwener
jackwener merged commit 1ec3107 into main Aug 3, 2026
9 of 11 checks passed
jackwener added a commit that referenced this pull request Aug 3, 2026
Premium-checklist follow-up to #2012 (the TSX half): the market-card
category chip repeated a neutral Badge on every card — the docs'
named DON'T. Categories are tags, and Token is the tag component;
the exception-only warning Badges on library rows (state_error /
needs-review / context attention) stay Badges by the same rule.

Self-reviewed per the standing instruction. Verified: build,
typecheck, format:check, dead-css, checks, Storybook smoke (74
renders x 3 viewports).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant