Design pass — public pages: theme, shared shell, landing rework - #37
Draft
ArneeMe wants to merge 1 commit into
Draft
Design pass — public pages: theme, shared shell, landing rework#37ArneeMe wants to merge 1 commit into
ArneeMe wants to merge 1 commit into
Conversation
Theme upgrade (refined green palette, muted amber secondary, system font stack with tighter headings, rounded pill buttons, soft shadows, styled anchors) plus CssBaseline so the background and typography defaults actually apply. New shared PublicShell header/footer for the landing, /om and /personvern pages — single language toggle in the header, standard links in the footer. Landing rebuilt: centered hero, numbered step cards, accent privacy card, outlined org chips. The org-branded pages (form, verify) keep their own headers and inherit the theme. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01BFHkXUuHJME4cF9AJWbavB
Deploying inf319-certificate with
|
| Latest commit: |
975f810
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://7321634a.inf319-certificate.pages.dev |
| Branch Preview URL: | https://claude-review-13-design.inf319-certificate.pages.dev |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Last part of the review stack (on the SEO PR). Pure presentation — no logic, no strings changes, no security surface. Merge this last; everything upstream merges to main first, then this makes it pretty.
What changed
customTheme.ts): refined deep-green palette (primary doubles as the verify page's "valid" color, so it stays semantic), muted amber secondary instead of pure orange, system font stack with tighter/heavier headings, pill buttons, soft shadows, off-white page background, styled anchor links.CssBaselineadded in the root layout — the app never had it, which is why everything looked like unstyled MUI defaults.PublicShell(new): shared slim header (wordmark + single language toggle) and footer (om / personvern / admin login / GitHub) for the landing,/omand/personvern. The org-branded pages (form, verify) deliberately keep their own headers — the org's identity leads there./omand/personvernwrapped in the shell; their duplicate language toggles removed.Verification
Typecheck, lint, 110 unit tests, all 8 Playwright E2E (landing heading, admin-login link, both language-toggle walks) and production build pass. Check the Cloudflare preview on this PR to see the result live.
🤖 Generated with Claude Code
https://claude.ai/code/session_01BFHkXUuHJME4cF9AJWbavB
Generated by Claude Code