Redesign public website (landing + status)#48
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b4c2adc18e
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| @media (prefers-color-scheme: light) { | ||
| :root { | ||
| color-scheme: light; | ||
| --bg: #f7f3ed; |
There was a problem hiding this comment.
Preserve AA contrast in light mode
For users whose system prefers light color scheme, this new light background still uses the dark-theme --brand value #f26430 for .kicker, .brand em, and the hero highlight. The repo's WCAG contrast helper gives only 2.86:1 for #f26430 on #f7f3ed, below the 4.5:1 AA requirement for the small/normal text used here, so the page becomes hard to read in light mode unless --brand is overridden or the light theme is removed.
Useful? React with 👍 / 👎.
| } | ||
| .nav-inner { min-height: 72px; display: flex; align-items: center; justify-content: space-between; gap: 24px; } | ||
| .brand { display: inline-flex; gap: 10px; align-items: center; text-decoration: none; font: 750 16px var(--mono); letter-spacing: -0.02em; } | ||
| .logo { width: 34px; height: 34px; border-radius: 11px; display: grid; place-items: center; background: linear-gradient(135deg, var(--brand), var(--brand-2)); color: #111827; box-shadow: 0 14px 40px rgba(242, 100, 48, 0.28); } |
There was a problem hiding this comment.
Keep the redesign within the UI gate
In this redesigned stylesheet, node src/cli.js uicheck design landing/index.html --json exits 1: it reports 7 radius levels (max 3) and 16 colors (max 8), while the parent version passed the same check. The hard-coded radii/colors introduced in this area (11px here plus 10/14/16/18/24/999 elsewhere) break the documented public-site design gate, so release checks that run forge uicheck design will reject the page until these are consolidated into the design tokens.
Useful? React with 👍 / 👎.
Motivation
Description
landing/index.htmlwith a modern, responsive design, upgraded metadata/OG/schema, light/dark mode, reduced-motion support, and a stronger hero, CTA, metric strip, platform cards, workflow, evidence, and data-sources sections.Skip to contentlink, semantic landmarks (header/main/section), visible focus states, keyboard-friendly copy interaction for install command, and mobile breakpoints.public/index.htmlfrom repo data using the existing pages build pipeline (scripts/build-pages.mjs→npm run pages:build) and committed the updated generated page (commit:d60766a).Testing
npm run checkwhich completed (Biome warnings unrelated to these edits were reported).npm testwhich passed:605tests passed and2skipped.npm run typecheckwhich completed without errors.npm run pages:buildwhich regeneratedpublic/index.htmlsuccessfully.node src/cli.js uicheck design landing/index.html public/index.htmlwhich exited normally but reported strict design-gate failures forpalette-sizeandradius-levels(style consolidation warnings).node src/cli.js uicheck visual landing/index.htmlwhich was skipped because Playwright is not available in this environment.Codex Task