-
Notifications
You must be signed in to change notification settings - Fork 13
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
Remove Sass #1
Comments
Does this issue mean to remove Sass entirely? What would be the benefit to that? Most of the issues with complexity were fixed in the CSS refactor, and we're still using Sass features like mixins and extends. |
Originally we planned on using PostCSS for the project; using SASS was just an artifact of BlockBase. Using PostCSS would make it consistent w/ the global header and be more modern. Integrating w/ Gutenberg dependencies might be tricky though. |
I'm going to transfer this issue over to the parent theme repo — i think removing Sass from that theme & future children will be easier than trying to remove it from here, where we're so tied in to mixins & extends (we can always reconsider the idea if/when we make News a proper child of that parent). |
ideally, the theme would use postcss like
wporg-mu-plugins
does.xref WordPress/wporg-mu-plugins#6 (review)
the sooner we do it, the easier it'll be, but i'm also leery of getting bogged down in non-essentials right now 🤔
8270ecd1f04223c53a9ffcc6a81fa118f5ba8a09 added
%footer-archive-dark
,%local-header-off-white
, etc. If PostCSS doesn't have a similar concept, we might be able to convert them to regular classes, and have the templates add the class, but that would rely on WordPress/gutenberg#32939we currently import some mixins, breakpoints, etc from gutenberg's sass. i'm not sure a good solution for that
there may be a tool that'll convert most sass to postcss automatically
The text was updated successfully, but these errors were encountered: