Fix CSScomb when Sass tmLanguage installed #52

Merged
merged 1 commit into from Feb 3, 2015

Projects

None yet

2 participants

@karlhorky
Contributor

CSScomb was not running when Sass tmLanguage was used for syntax highlighting.

Fixes #18

@tonyganch tonyganch commented on an outdated diff Feb 3, 2015
CSScomb.py
@@ -104,6 +104,7 @@ def is_css(self):
def is_scss(self):
return self.view.scope_name(0).startswith('source.scss')
+ or self.view.scope_name(0).startswith('source.sass')
@tonyganch
tonyganch Feb 3, 2015 Member

Oh, wow, thank you!
But you know what, there should actually be def is_sass() returning sass as a syntax name, not scss.

@karlhorky karlhorky Fix CSScomb with Sass tmLanguage
The Sass tmLanguage package can be used to highlight SCSS and Sass files. If
used on an SCSS file, CSSComb does not run (since it does not match the scope
name as starting with `source.scss`).

This commit enables CSSComb for both Sass and SCSS files when being highlighted
with the Sass tmLanguage package.

Fixes #18
14e3792
@karlhorky
Contributor

@tonyganch No problem, glad to help :)

Please review again. I've added is_sass() as well to support Sass files as well as SCSS files using the Sass tmLanguage package.

@tonyganch tonyganch merged commit 9b6467d into csscomb:master Feb 3, 2015
@tonyganch
Member

Thank you! 🐘

@karlhorky karlhorky deleted the unknown repository branch Feb 3, 2015
@karlhorky
Contributor

Thanks for the merge :bowtie:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment