chore(fonts): remove SharpSans - #80
Conversation
WalkthroughReplaces the custom ChangesFont Migration: SharpSans → Plus Jakarta Sans
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
✅ Deploy Preview for loadsmart-engineering ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
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 `@gatsby-config.js`:
- Around line 87-90: The Google Fonts request in the Gatsby config is missing
the 400 weight for Plus Jakarta Sans, so normal body text cannot load the
intended face. Update the fonts entry in the Gatsby config’s fonts array to
include the 400 weight alongside the existing Plus Jakarta Sans weights, keeping
the request consistent with the body copy’s font-weight 400.
🪄 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: Repository: loadsmart/coderabbit/.coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: e3a1cbac-f160-445e-9fc4-5323eac3b530
⛔ Files ignored due to path filters (16)
src/styles/fonts/SharpSans-Bold.eotis excluded by!**/*.eotsrc/styles/fonts/SharpSans-Bold.woffis excluded by!**/*.woffsrc/styles/fonts/SharpSans-Bold.woff2is excluded by!**/*.woff2src/styles/fonts/SharpSans-Book.eotis excluded by!**/*.eotsrc/styles/fonts/SharpSans-Book.woffis excluded by!**/*.woffsrc/styles/fonts/SharpSans-Book.woff2is excluded by!**/*.woff2src/styles/fonts/SharpSans-Light.eotis excluded by!**/*.eotsrc/styles/fonts/SharpSans-Light.woffis excluded by!**/*.woffsrc/styles/fonts/SharpSans-Light.woff2is excluded by!**/*.woff2src/styles/fonts/SharpSans-Medium.eotis excluded by!**/*.eotsrc/styles/fonts/SharpSans-Medium.woffis excluded by!**/*.woffsrc/styles/fonts/SharpSans-Medium.woff2is excluded by!**/*.woff2src/styles/fonts/SharpSans-Semibold.eotis excluded by!**/*.eotsrc/styles/fonts/SharpSans-Semibold.woffis excluded by!**/*.woffsrc/styles/fonts/SharpSans-Semibold.woff2is excluded by!**/*.woff2yarn.lockis excluded by!**/yarn.lock,!**/*.lock
📒 Files selected for processing (6)
gatsby-config.jspackage.jsonsrc/components/Layout.jssrc/components/Title.jssrc/styles/theme.jssrc/styles/typography.css
💤 Files with no reviewable changes (3)
- src/styles/typography.css
- src/components/Layout.js
- package.json
Types of changes
Description of the proposed changes
Remove SharpSans typefaces
Screenshots
Summary by CodeRabbit