Skip to content

Public shell: framed profile + certificate verification pages (batch unit 5) - #31

Merged
duckyquang merged 1 commit into
mainfrom
feat/batch-5-public-shell
Jul 13, 2026
Merged

Public shell: framed profile + certificate verification pages (batch unit 5)#31
duckyquang merged 1 commit into
mainfrom
feat/batch-5-public-shell

Conversation

@duckyquang

Copy link
Copy Markdown
Member

Summary

  • PublicProfile (/p/:key): white hairline top-nav (wordmark + Archive/dashboard links) on the --bg-muted canvas, centered 760px column of white hairline cards, uppercase micro-labels (.pp-label) heading each section, hairline-separated publication rows. The not-found state moved off the legacy .login-wrap chrome onto the new standalone .pv-wrap shell.
  • VerifyCertificate (/verify-certificate): rebuilt off the legacy sky-gradient .login-wrap/.login-card (and gold .yellow-text heading) onto a clean centered standalone page — brand mark + Inter heading, white hairline card with a labelled input, success/danger result states via the existing guide-ico-success/danger tones, muted footer link. ?code= auto-verify preserved.
  • styles.css (unit-5 regions only): archive shell retokenized and .archive-hero (unused in any JSX) deleted; .arx-* rows tokenized (--bg/--line/--radius-lg/--shadow-md/--font-mono) which let the hardcoded dark-mode overrides go; new .pv-*/.pp-* rules appended under /* === unit-5: public-shell === */.
  • design-system.css: the .archive-* !important overrides collapsed into the one clean definition — no !important needed.
  • Certificates.jsx: preview image chrome moved to tokens (radius/hairline/shadow); the Cinzel canvas rendering is untouched.

Testing

  • Backend suite: 201/201 passing.
  • E2E (backend :4005, vite :5205): /p/sam and /verify-certificate empty + genuine (SYN-… issued via demo login) + not-found states, light + dark, 1440px + 375px — all verified in-browser.

Notes for coordinator

  • Impeccable hook flags 3 pre-existing findings in sibling regions of styles.css (progress-bar width transitions ~L712/L1764, DM reply side-border ~L1912) — left untouched per region ownership.
  • Hook also flags Inter as an "overused font" in design-system.css — that is the mandated system typeface; false positive.

🤖 Generated with Claude Code

- PublicProfile: white hairline top-nav on --bg-muted canvas, centered
  column of white hairline cards, uppercase micro-labels (.pp-label),
  hairline-separated publication rows; not-found state moved off the
  legacy login chrome onto the new standalone .pv-wrap shell
- VerifyCertificate: rebuilt off the legacy sky-gradient .login-wrap
  onto a clean centered .pv-wrap page — brand mark, Inter heading,
  white hairline card, labelled input, success/danger result states
  via guide-ico tones, muted footer link
- styles.css: archive shell retokenized (--bg-muted canvas, hairline
  nav), unused .archive-hero deleted, .arx-* rows tokenized
  (bg/line/radius/shadow/mono tokens) so dark overrides could go
- design-system.css: collapsed the .archive-* !important overrides
  into the one clean definition in styles.css
- Certificates: preview image chrome on tokens (radius/line/shadow);
  canvas rendering untouched

Co-Authored-By: Claude Fable 5 <[email protected]>
@vercel

vercel Bot commented Jul 12, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
dashboard Ready Ready Preview, Comment Jul 12, 2026 9:09am

@duckyquang
duckyquang merged commit 1b668a1 into main Jul 13, 2026
3 of 4 checks passed
duckyquang added a commit that referenced this pull request Jul 13, 2026
…ration tree

Those four unit branches are already fully contained in this branch with
their conflicts resolved against the unit-1 foundation (which main does not
have yet). main introduced no other content, so the integrated tree is kept
as-is.

Co-Authored-By: Claude Fable 5 <[email protected]>
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