-
-
Notifications
You must be signed in to change notification settings - Fork 9.1k
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
Ndelangen/remove css usage #9003
Conversation
This pull request is being automatically deployed with ZEIT Now (learn more). 🔍 Inspect: https://zeit.co/storybook/monorepo/vq640ga9x |
I did this be cause I was investigating the inclusion of all css of all languages of syntax-highlighter. Though I wasn't able to reproduce #8972 I did see huge blobs of string in our bundle. I figured it would save some bundle-size, because these blobs of string contained all sorts of comments that cannot be removed, but now they can. Packtracker seems to agree this is an improvement, though smaller than I had hoped. |
Issue: these styled.div`` string stay in the bundle as is, they cannot be minified, increasing the size of the bundle, especially in production mode.
What I did
I migrated them to use object notation