MarioRicalde / SCSS.tmbundle

The TextMate SCSS Official Bundle. Now Compatible with SublimeText2.
834 stars 110 forks source link

Keyword Syntax Scope? #135

Closed bobrocke closed 11 years ago

bobrocke commented 11 years ago

This SCSS bundle uses keyword.other.unit.scss. The LESS bundle uses keyword.unit.css. The official CSS distribution uses keyword.unit.css.

Should this bundle change to keyword.unit.css?

infininight commented 11 years ago

Actually the CSS bundle uses keyword.other.unit.css as well, this is correct according to the naming conventions.

infininight commented 11 years ago

Went ahead and fixed the LESS bundle with textmate/less.tmbundle@041259f75b3ce23e59aa1d2c7da9672a183e0845.

bobrocke commented 11 years ago

Hmm. My CSS Bundle does this:

                {   name = 'constant.numeric.css';
                    match = '(?x)
                        (?:-|\+)?(?:(?:[0-9]+(?:\.[0-9]+)?)|(?:\.[0-9]+))
                        ((?:px|pt|ch|cm|mm|in|r?em|ex|pc|deg|g?rad|dpi|dpcm|s)\b|%)?
                    ';
                    captures = { 1 = { name = 'keyword.unit.css'; }; };
                },

Maybe I have an old/wrong one. I've replaced it.