cipherdevgroup / cipher-theme-config

Grunt configuration for Cipher themes
MIT License
8 stars 4 forks source link

Consider bringing back grunt-wp-css for a final build task #10

Closed robneu closed 8 years ago

robneu commented 8 years ago

This was suggested by @ozzyrod as an option to make the final CSS compliant with WP coding standards. It was removed to allow sourcemaps to work, but an additional build alias that can be used as part of the package task could be nice to give people a cleaner CSS source file to work with.

I'm not 100% on this, because I'm really not sure it matters. People really shouldn't be editing the generated CSS very much if at all. If they want to customize it, they should use the Sass files or if they're not comfortable with that they can create a child theme or use a custom CSS plugin.

Leaving this open for discussion.

robneu commented 8 years ago

This is accomplished with the grunt build:dist task which is now used in the grunt package:dist task.

Themes which need to utilize this config will need to update their theme aliases to include this configuration.