-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Move fonts to reduce bundle size #19
Conversation
- moved the font files to the public folder - changes the typography css rules to point to them so that they can be served as-is without being base64'd and added inline to style.css on build This might solve a large portion of #15 but we should leave the issue open to tackle this a little deeper even if it does
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@leo95oliveira vercel isnt able to find the fonts in the public folder in the preview for some reason? This is specific to either vercel or storybook config as the build runs fine locally. Any ideas? |
Looking at the build logs on vercel, I think the issue is pertaining to Storybook. I'm not 100% sure but I'll take a look later this afternoon and see what I can find! |
- fixed the storybook issue with not picking up absolute paths by using a webpack config specifically in the storybook config (so it doesnt affect/change the library itself)
@leo95oliveira fixed the issue with storybook (it was an issue with how absolute urls are handled) if you review this and its all good, should we bundle it in with the v2.0.0 release? |
Legend! Looks great! Thank you for this! Yes, lets bundle it with v2! |
@leo95oliveira no rush, just checking if you intended to approve this one as your comments suggested you did but its still pending review 🙏 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My apologies @SamusElderg! I thought I had already approved this 😭 thank you for tagging me
This might solve a large portion of #15 but we should leave the issue open to tackle this a little deeper even if it does