changcheng / wro4j

Automatically exported from code.google.com/p/wro4j
0 stars 0 forks source link

SassCssSupport doesn't handle spaces instead of tabs #456

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. configure wro4J to use the SassCssSupport preprocessor
2. create a .scss file for use with the SassCssSupport preprocessor
3. add it to a group as a CSS resource
4. request that resource

What is the expected output? What do you see instead?
Expected output is the resulting CSS file after being processed by 
SassCssSupport.  Result is an exception being thrown: Error: ParseError: on 
line 5; invalid indentation, to much nesting (sass-0.5.0.min.js#1)

What version of the product are you using? On what operating system?
1.4.6 OSX 10.6.8, Java 1.6.0_31

Please provide any additional information below.
Replacing four spaces (used by our team) with tabs alievitated this error

Original issue reported on code.google.com by aaron.si...@opower.com on 7 Jun 2012 at 4:57

GoogleCodeExporter commented 9 years ago

Original comment by alex.obj...@gmail.com on 7 Jun 2012 at 7:54

GoogleCodeExporter commented 9 years ago
Don't know if the fix is the best one, since I just patched the sass.js by 
removing the error check for indentation (which doesn't make much sense right 
now): 

the following line was commented out:  error('invalid indentation, to much 
nesting')

Original comment by alex.obj...@gmail.com on 15 Jun 2012 at 9:49

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
If you still  can reproduce this in 1.4.8, could you provide a SASS resource 
which causes the problem? This would help to use as a test-case. 

Original comment by alex.obj...@gmail.com on 10 Aug 2012 at 6:46