Fix missing social cards across Suede DNA pages - #2
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 279becccbe
ℹ️ 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".
|
|
||
| <g transform="translate(76 68)" | ||
| font-family="'SFMono-Regular', Consolas, 'Liberation Mono', monospace"> | ||
| <text x="0" y="0" font-size="21" font-weight="700" letter-spacing="5" fill="#22d3ee">SUEDE / DNA</text> |
There was a problem hiding this comment.
Reserve cyan for the signal trace
The social card colors the passive SUEDE / DNA wordmark cyan, even though cyan is reserved exclusively for live/active signal. Keep the wordmark in a neutral archival color and use cyan only for the waveform/strand, then regenerate the referenced WebP so the shipped card matches the source.
AGENTS.md reference: AGENTS.md:L24-L27
Useful? React with 👍 / 👎.
Adds one compressed 1200x630 Suede DNA share card and explicitly preserves it across all docs and article metadata overrides. Includes an enumerated regression test for the 11 affected pages. Verification: typecheck passed; lint passed; 92 tests passed; production build generated all 410 static pages.