Repository navigation
Conversation
Closes #303. - Code samples on /keys go through a CodeBlock: a bordered block that scrolls long lines instead of wrapping them, with a copy button. Inline code gets its own style. - The header has a Faucet link next to For Agents, with aria-current on the page you are on. On a phone the nav takes its own row. - The footer link back from /keys and the header both read one KEYS_NETWORK constant. - The claim card and the keys card share max-w-lg. - Info cards: a heavier heading with a bar in the brand colour, body text no longer capped at 30ch, tighter heading-to-body spacing. - The grid uses equal shares of a capped width. A long code line used to widen its column and push /keys 238px past a 375px screen. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Collaborator
Author
|
Checked on the Vercel preview (https://faucet-git-ghozzza-303-faucet-cards-and-code-c-labs.vercel.app), with the same Playwright measurement as the body. |
ghozzza
force-pushed
the
ghozzza/303-faucet-cards-and-code
branch
from
October 30, 2026 15:38
fc660b9 to
bb8af34
Compare
This branch was successfully deployed
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
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.
Description
Styles the claim page's info cards and the code samples on
/keys, and puts a way back to the faucet in the header. No copy, API, auth or claim-flow changes./keys. A newCodeBlock(components/code-block.tsx) gives the three snippets a bordered block in the secondary background, with a Copy button. Long lines scroll inside the block and never wrap, so the curl command and the status URL stay intact. The status snippet usedwhitespace-pre-wrap break-alland now scrolls like the others. Inlinecodegets its own small boxed style inglobals.css, so401 invalid_api_keyno longer reads as prose.aria-current="page"and is highlighted in the brand colour. On a phone the nav takes its own row under the logo, because two links, sign-in and the toggle do not fit beside it in 375px./keys. The footer link and the header both read oneKEYS_NETWORKconstant. Keys are Celo Sepolia only, so that is the page's network.max-w-lg.Other changes
/keyswas 613px wide on a 375px screen, light and dark, measured onmaster. The grid'sminmax(25%, auto)/1frcolumns took a long code line's width as their minimum. Columns are nowminmax(0, 1fr)shares of a grid capped at 80rem, and cards getmin-width: 0. This also makes the grid the same width on both pages./celo-sepoliaat 1920px had no horizontal overflow onmaster, and it has none now.Tested
yarn --cwd apps/web test:ci: 228 passed, 8 of them new (tests/faucet-ui.test.ts, plus two intests/faucet-header.test.ts). Like the existing header test they assert the source, since there is no DOM test setup. Run againstmaster, all 8 fail and the 5 existing header tests still pass.yarn lint,tsc --noEmit,next buildand prettier on the touched files are clean.scrollWidth > clientWidthwith Playwright on production builds (next build && next start) ofmasterand this branch, both pages, light and dark, at 375, 1024, 1440 and 1920.masterhas 2 of 16 (/keysat 375, light and dark). This branch has 0 of 16./celo-sepolia/celo-sepolia/celo-sepolia/celo-sepolia/keys/keys/keys/keysNot tested here: the signed-in state of
/keys(key list, fresh-key box), since that needs a GitHub OAuth app. Those elements are unchanged apart from inheriting the inlinecodestyle for the fresh key.Open questions from the issue, answered
--main,--secondary-background,--border, the 2px borders and offset shadows) rather than introducing new colours.Related issues
Backwards compatibility
Styling and markup only. No API, route or data change.
Documentation
None needed.
🤖 Generated with Claude Code