SassyStudio icon indicating copy to clipboard operation
SassyStudio copied to clipboard

Absent line terminator after variable, recompile fails

Open blackf0rk opened this issue 11 years ago • 5 comments
trafficstars

screen shot 2013-12-26 at 12 41 14 pm

If you forget to add a semicolon to the end of your line after a variable, the .css isn't recompiled. It seems to only fail after variables.

blackf0rk avatar Dec 26 '13 18:12 blackf0rk

Yeah, you should see an error about it in the output window in the SassyStudio pane. The error is coming from libsass and you would likely see this error with other precompilers (ruby sass gem, compass gem) as well. I'll add some error integration into the extension so that you get some red squigglies there, but I don't know if there's anything else I can do.

If the problem is mainly that you don't notice when this happens, there's an option that will replace the css file with the exception so that you can easily notice that there was a problem as everything will lose it's styling.

darrenkopp avatar Dec 26 '13 19:12 darrenkopp

@darrenkopp I see the error in the output window now, thanks. I also found that if you reference a variable that doesn't exists, the css doesn't compile either. I think it would be helpful to have syntax highlighting under elements that would cause a compilation to fail.

I can't speak for other developers, but I don't keep my output window open, so it would be helpful if there was, at the very least, some sort of warning/alert if the css fails to compile. As soon as I would save the .scss file I would get that warning, and I would know that whatever it was I was working on last is wrong; as opposed to working with the file, saving, thinking nothing is wrong and then 30 lines later realize it's not compiling.

blackf0rk avatar Dec 26 '13 19:12 blackf0rk

Yeah I haven't been able to figure out how to get the output window to pop open when I'm telling it to do that, but for now I'd definitely tick the 'Replace .css with exception' option in Tools > Options and Setings > Sassy Studio > General. This will make exceptions far more apparent when they happen in the meantime.

darrenkopp avatar Dec 26 '13 19:12 darrenkopp

That's good enough for me, thanks! :+1:

blackf0rk avatar Dec 26 '13 19:12 blackf0rk

Cool, I'll revisit how close I am for error / warning integration. When that is done, I'll close this issue with that as the resolution.

darrenkopp avatar Dec 26 '13 19:12 darrenkopp