feat(site): switch display type to Instrument Sans#499
Conversation
…ar on /components Replace Young Serif with self-hosted Instrument Sans, share docs shell layout with the components index, fix category grid typography, and update OG cards. Co-authored-by: Cursor <cursoragent@cursor.com>
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThis PR consolidates typography system changes and layout refactoring across the documentation and components sections. A new ChangesTypography System and Layout Consolidation
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Use pl-5.5 so the section label lines up with chevron-aligned category titles. Co-authored-by: Cursor <cursoragent@cursor.com>
Fixes frozen-lockfile CI failure from the Instrument Sans migration. Co-authored-by: Cursor <cursoragent@cursor.com>
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@CLAUDE.md`:
- Line 20: CLAUDE.md currently references a non-existent spec file
animata/skeleton/CATEGORY-GLYPHS.md; either update that link to the actual spec
location or add the missing markdown spec. Locate the reference in CLAUDE.md and
replace animata/skeleton/CATEGORY-GLYPHS.md with the correct path to the spec
(or create a new CATEGORY-GLYPHS.md in animata/skeleton/), and ensure it aligns
with the existing implementation file animata/skeleton/category-glyphs.tsx and
any related stories/styles so contributors can follow the guideline.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: f1b2e629-f36e-405a-9937-5465eb4564bb
⛔ Files ignored due to path filters (2)
public/og-fonts/instrument-sans-latin-wght-normal.woff2is excluded by!**/*.woff2public/og-fonts/young-serif-latin-400-normal.woff2is excluded by!**/*.woff2
📒 Files selected for processing (12)
CLAUDE.mdapp/(main)/components/layout.tsxapp/(main)/components/page.tsxapp/(main)/docs/[[...slug]]/page.tsxapp/(main)/docs/layout.tsxapp/layout.tsxcomponents/docs-shell-layout.tsxpackage.jsonscripts/lib/og-template.mjsscripts/sync-og-fonts.mjsstyles/docs.cssstyles/fonts.css
💤 Files with no reviewable changes (1)
- package.json
Replace the stale animata/skeleton/CATEGORY-GLYPHS.md reference with the contributing guide paths and document the real implementation files. Co-authored-by: Cursor <cursoragent@cursor.com>
and add docs sidebar on /components
Replace Young Serif with self-hosted Instrument Sans, share docs shell layout with the components index, fix category grid typography, and update OG cards.
Summary by CodeRabbit
Style
New Features
Documentation
Chores