A small patch that includes a config.rb file, so theme editors don't have to create one.
It also renames scss/mixins.scss to scss/_mixins.scss
and scss/variables.scss to scss/_variables.scss
(and updates scss files that reference them.
This stops compass from generating css files from these include files.

I intend to release another patch that puts all the theme colours in the variables.scss file as sass variables. That way new themers just need to make changes in one place to update their entire theme.
Please let me know if you don't think this is worthwhile doing.

CommentFileSizeAuthor
commerce_kickstart_sass.patch8.97 KBmedden

Comments

bojanz’s picture

bojanz commented
Status: Active » Needs review

Sounds great!

Setting issue status to "needs review" so that someone can commit the first patch later today.

dudenhofer’s picture

dudenhofer commented
Status: Needs review » Reviewed & tested by the community

Nice patch, I would just disable the line comments and set the output style to :nested to avoid re-writing all of the CSS files every time some one switches between compass watch and sass --watch

dudenhofer’s picture

dudenhofer commented
Status: Reviewed & tested by the community » Needs work

Sorry, should have marked this differently. But a simple fix ^.

heyyo’s picture

heyyo commented

If I'm starting from a fresh sub-theme of Omega Kickstart, would I be able to use those SASS files ? Or those files are only usable for overriding the Commerce Kickstart theme ?

dudenhofer’s picture

dudenhofer commented
Status: Needs work » Needs review

I edited the config so it won't completely rewrite all the css files from previous commits.
https://code.drupalcommerce.org/#/c/685/

jsacksick’s picture

jsacksick commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.