fix(docs-og-image): match reference cover template typography exactly#5598
Conversation
- swap Season Sans for Söhne Kräftig (500) — the reference cover's actual brand font, confirmed by letterform comparison; recovered from git history since it was removed as an unused static asset - fix ink/background colors to exact reference hex values - square caps + miter join on the corner arrow to match the reference's sharp corners instead of rounded ones - recalibrate title font size, line height, and wrap width for the new font's metrics
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryLow Risk Overview Typography and colors: Title font switches to Söhne Kräftig (500) via Title wrapping: Line packing no longer uses a flat Latin character count; it estimates width with separate Latin vs CJK em widths and can split oversized “words” (common for Reviewed by Cursor Bugbot for commit 962b943. Configure here. |
Greptile SummaryThis PR updates the docs OG image to match the reference cover template. The main changes are:
Confidence Score: 5/5This looks safe to merge.
Important Files Changed
Reviews (3): Last reviewed commit: "fix(docs-og-image): fall back to charact..." | Re-trigger Greptile |
…r-wrap wrapTitleLines budgeted a flat 0.42em/char, tuned for Latin text. Docs ships ja/zh locales — CJK glyphs render near-square (~1em), so a CJK title could overflow the fixed-width title box uncaught. Sum per-char em-width with a CJK-range check instead of counting characters.
|
@cursor review |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit 73b53cd. Configure here.
…JK words wrapTitleLines only splits at spaces, so a space-free CJK run (common for Chinese titles) still arrived as a single word wider than the title box and rendered as one overflowing line. Falls back to character-level chunking for any word that alone exceeds maxWidthEm.
|
@cursor review |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit 962b943. Configure here.
Summary
#515151/#c1c1c1)Type of Change
Testing
Tested manually — rendered the route locally and pixel-compared against the reference cover template (arrow, colors, wordmark, and title all matching within a few px).
Checklist