Merge pull request #27 from bolandrm/source_maps

Inline Source maps

Author: bolandrm

README.md

@@ -24,6 +24,19 @@ This should essentially be a drop in alternative to [sass-rails](https://github.
 
 __Note:  This is a new project, please report any issues you come across!__
 
+## Inline Source Maps
+
+With SassC-Rails, it's also extremely easy to turn on inline source maps. Simply
+add the following configuration to your development.rb file:
+
+```ruby
+  # config/environments/development.rb
+  config.sass.inline_source_maps = true
+```
+
+Note, as indicated, these source maps are *inline*.  They will not generate additional
+files or anything like that.  Instead, they will be appended to the compiled
+application.css file.
 
 ## LibSass Compatibility With Ruby Sass
 

lib/sassc/rails/template.rb

@@ -13,13 +13,12 @@ def call(input)
           syntax: self.class.syntax,
           load_paths: input[:environment].paths,
           importer: SassC::Rails::Importer,
-          style: sass_style,
           sprockets: {
             context: context,
             environment: input[:environment],
             dependencies: context.metadata[:dependency_paths]
           }
-        }
+        }.merge(config_options)
 
         engine = ::SassC::Engine.new(input[:data], options)
 
@@ -44,12 +43,11 @@ def evaluate(context, locals, &block)
           syntax: syntax,
           load_paths: context.environment.paths,
           importer: SassC::Rails::Importer,
-          style: sass_style,
           sprockets: {
             context: context,
             environment: context.environment
           }
-        }
+        }.merge(config_options)
 
         ::SassC::Engine.new(data, options).render
       end
@@ -61,9 +59,23 @@ def evaluate(context, locals, &block)
       include Sprockets2
     end
 
+    def config_options
+      opts = { style: sass_style }
+
+
+      if Rails.application.config.sass.inline_source_maps
+        opts.merge!({
+          source_map_file: ".",
+          source_map_embed: true,
+          source_map_contents: true,
+        })
+      end
+
+      opts
+    end
+
     def sass_style
-      style = Rails.application.config.sass.style || :expanded
-      "sass_style_#{style}".to_sym
+      (Rails.application.config.sass.style || :expanded).to_sym
     end
   end
 

sassc-rails.gemspec

@@ -26,7 +26,7 @@ Gem::Specification.new do |spec|
   # unfortunately we require sass for now, so that we can
   # reuse portions of the sprockets template
   spec.add_dependency 'sass'
-  spec.add_dependency "sassc", "~> 1.4"
+  spec.add_dependency "sassc", "~> 1.5"
 
   spec.add_dependency "tilt"
 

test/sassc_rails_test.rb

@@ -221,6 +221,16 @@ def test_sassc_compression_is_used
     SassC::Engine.expects(:new).with("", {style: :compressed}).returns(engine)
     render_asset("application.scss")
   end
+
+  def test_allows_for_inclusion_of_inline_source_maps
+    @app.config.sass.inline_source_maps = true
+    initialize_dev!
+
+    asset = render_asset("application.scss")
+    assert_match /.hello/, asset
+    assert_match /sourceMappingURL/, asset
+  end
+
   #test 'sprockets require works correctly' do
   #  skip