Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Revert "Use sass compressor as the default CSS compressor"
This reverts commit 3eef1c7.
The css compressor should be set in sass-rails plugin. Check
rails/sass-rails@339529f
for the fix.
  • Loading branch information
guilleiguaran committed Feb 25, 2013
1 parent 3eef1c7 commit ebe9590
Showing 1 changed file with 1 addition and 1 deletion.
Expand Up @@ -25,7 +25,7 @@
<%- unless options.skip_sprockets? -%> <%- unless options.skip_sprockets? -%>
# Compress JavaScripts and CSS. # Compress JavaScripts and CSS.
config.assets.js_compressor = :uglifier config.assets.js_compressor = :uglifier
config.assets.css_compressor = :sass # config.assets.css_compressor = :sass


# Whether to fallback to assets pipeline if a precompiled asset is missed. # Whether to fallback to assets pipeline if a precompiled asset is missed.
config.assets.compile = false config.assets.compile = false
Expand Down

3 comments on commit ebe9590

@teddyzetterlund
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If this is being done by the sass-rails plugin, why is it only commented out in this file instead of removed?

@guilleiguaran
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@teddyzetterlund this is left just as a guide for users that want to switch to another compressor

@teddyzetterlund
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@guilleiguaran Makes sense, though I think it kind of implies that it's not being done already (by default) by sass-rails. Maybe it's worth adding some clarity somehow?

Please sign in to comment.