Make compression style configurable to ensure compatibilty with Webkit

[rails]

Make compression style configurable to ensure compatibilty with Webkit

[chriseppstein]

There needs to be:

1. A webkit bug filed/referenced about this issue so that we can understand it and track it.
2. A bug filed against Sass, not sass-rails to address any line-length issues in :compressed mode.

For sass-rails it sounds like the only issue here is that the config.sass.style setting is not being respected by the compressor.

Is this a fair assessment?

[colszowka]

Hi Chris, yes, that would be sufficient :)

[chriseppstein]

Great. Can you please file a new bug against sass with the details?

[chriseppstein]

Ok I think I might know what is going on here.

At one point in webkit's history they had a parsing bug if you used @charset anywhere but the first line of a file, they would drop everything to the end of the line. It is part of the spec that @charset is only legal on the first line. So to correct this, sass strips out any charset directive and sets a single charset at the top of the file. Perhaps you're using sprockets to concatenate many sass files resulting in triggering this @charset directive bug? If so, you need to file a bug against sprockets to handle this. Otherwise, don't use sprockets require and only use sass import directives.

Please reopen if I have misdiagnosed.

[ekaitzht]

@colszowka could you fix this problem? It's happening to me the same.

[colszowka]

@ekaitzht Sorry, not really. I think I moved over to doing sass include statements instead of requires, that usually is much better to work with than using sprockets anyway (it also works with plain css files, for example when you import them from bower components), so this went away. Originally I had changed the sass compression method to not compress though, as described in the initial post.

[ekaitzht]

@colszowka ok thanks