-
Notifications
You must be signed in to change notification settings - Fork 42
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
Autoprefix / prefixfree transform #64
Comments
👍 I'd definitely like to see a way to apply autoprefixer directly within the assetgraph compilation step. |
Should be pretty straight forward. The difficult thing about it is how to expose the configuration, which is far more complex than any of the really simple command line switches we have reduced configs down to before. We could just add the transform and feed it the js object it expects, and then not expose that setting in the command line tool, but for example have grunt-reduce pass it on as a config, since this is native js to begin with |
Note to the implementer: The CSS assets most likely need to be repopulated after the transform has run, as added or removed properties may contain relations to other assets. |
👍 Just have a sane default and most people won't need to change it ;) |
Looking into it now |
Released in assetgraph-builder v1.8.5 and grunt-reduce v0.1.13 |
We should make a transform that uses autoprefixer to clean up and improve css during a production build. See more in this discussion: sindresorhus/sublime-autoprefixer#3
I suggest having a transform that can be hidden behind a single
--prefix
option.I'm thinking of this behavior when
--prefix
is set.The only thing I haven't figured out is how to configure autoprefixer. Could possibly be a command line argument, but it seems it's to highly configurable for us to be able to map it directly. And a config file doesn't really fit the assetgraph paradigm that well at the moment.
Maybe reduce the config to some predefined options that can be chosen between? This would make configuration simpler, but take away power from the user.
The text was updated successfully, but these errors were encountered: