-
Notifications
You must be signed in to change notification settings - Fork 32
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
base: main
Are you sure you want to change the base?
Conversation
7acd21a
to
d9e9f6c
Compare
This does not work yet. Maybe we need to use postcss-purgecss plugin instead |
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. |
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. |
Trying to fix #3