fix: preload Inter font and use font-display: optional#788
fix: preload Inter font and use font-display: optional#788franklinjavier wants to merge 1 commit intoTanStack:mainfrom
Conversation
…t reflow CLS Preload the latin Inter woff2 so the font is available before first paint. Switch font-display from swap to optional so the browser never swaps in a late-arriving font, eliminating text reflow shifts.
👷 Deploy request for tanstack pending review.Visit the deploys page to approve it
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThe changes optimize font loading performance by adding an Inter font preload link to the root route's head configuration and adjusting the font-display CSS property from Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 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 |
Summary
font-displayfromswaptooptional— the browser uses the font if cached, otherwise sticks with the fallback, eliminating text reflow CLSTest plan
Summary by CodeRabbit