-
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
Only load the Foundation partials we need #36
Comments
Moving comments from Slack. This is a great suggestion and I'm glad you noted it. Typically during build, we include everything and then we slim it all down when we're done. We're using very little of Foundation ATM, but in the coming weeks, we'll need to clean up our list for sure. I just took a shortcut and did the one-liner to get us up and running. Feel free to champion this and get us a clear set of config and import files. We'll comment out what we don't need at the end. |
Dropping a reminder when we implement this that we should do the same with the Foundation JS. |
@sodevious I think we're at a point where you can run with this one now. |
My apologies if this is something already on your radar (probably is).
Towards the end of the project, we should make sure to create a manifest file that will only include the Foundation bits we are using and need to include, to avoid deploying unused CSS.
https://foundation.zurb.com/sites/docs/sass.html#adjusting-css-output
The text was updated successfully, but these errors were encountered: