search

thethemefoundry / forge

Build WordPress themes using Sass, LESS, and CoffeeScript
http://forge.thethemefoundry.com
MIT License
233 stars 51 forks source link

Add Theme Headers Config support #57

Closed jeffsebring closed 4 years ago

jeffsebring commented 11 years ago

One of the major reasons I like Forge, is the way it generated the theme headers in style.css.

When using css compression, comments are stripped.

Please add support for this again.

dstrojny commented 11 years ago

Good point Jeff! We added this to make it easier to package the Sass files on our end, but didn't take this into account. We'll look at maybe making this back in as an option. What do you think about something like?

config[:print_stylesheet_header] = true

We could comment that line out along with all the related arguments in the default config.rb file. This would make it easy to turn it on if you wanted to use it.

I can't guarantee when we're going to get around to working on this, but you're welcome to give it a shot and open a pull request if you like.

jeffsebring commented 11 years ago

That sounds like a good solution. I haven't had time to dive into this or the other issue, but will definitely do a pull request if I do.