Skip to content
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

Tree shaking css #4

Draft
wants to merge 10 commits into
base: main
Choose a base branch
from
Draft

Tree shaking css #4

wants to merge 10 commits into from

Conversation

aminya
Copy link

@aminya aminya commented Sep 22, 2020

Trying to fix #3

@aminya
Copy link
Author

aminya commented Sep 22, 2020

This does not work yet. Maybe we need to use postcss-purgecss plugin instead
https://v2.parceljs.org/languages/postcss/
https://github.com/FullHuman/purgecss/tree/master/packages/postcss-purgecss

@ryansolid
Copy link
Member

Yeah I'm not clear how these CSS cleaners work. The thing is the Realworld Demo uses pre-existing CSS files that aren't being included in the JS so I'm not sure a bundler has enough knowledge.

@aminya
Copy link
Author

aminya commented Sep 23, 2020

I have not been able to make this work. But if we do, that would be a huge benefit!

We might make a wrapper plugin specifically for Solid with the settings that we are sure would work.

@aminya aminya changed the title Tree shaking css using Purgecss Tree shaking css Sep 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tree shaking css
2 participants