Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

string scope? #8

Closed
pjv opened this issue Apr 27, 2012 · 2 comments
Closed

string scope? #8

pjv opened this issue Apr 27, 2012 · 2 comments

Comments

@pjv
Copy link

pjv commented Apr 27, 2012

i found something while trying to debug an issue in another ST2 package/plugin. not sure if this is wrong or not, but i thought i would bring it to your attention.

so, two test files; test.less and test.css each with the same code (cursor is where the pipe is):

body {
  background-image: url("|");
}

the output in the console for the command print (view.syntax_name(view.sel()[0].b))

test.css:
punctuation.definition.string.end.css string.quoted.double.css meta.property-value.css meta.property-list.css source.css

test.less:
variable.parameter.url support.function.any-method.builtin.css source.css.less

and as is apparent from the console output, syntax highlighting sees that position as a string in the .css file, but does not see it as a string in the .less file.

shouldn't that be a string in .less?

@thevasya
Copy link

thevasya commented Aug 2, 2012

This prevents AutoFileNames from working correctly on LESS files with ZenCoding installed which is pretty annoying.
Described here: liamcain/AutoFileName#8

Not sure if it's an issue with LESS-Sublime or with ZenCoding but @danro can you please take a look? Thanks!

@danro
Copy link
Owner

danro commented Nov 21, 2012

I've changed these to string.quoted.variable.parameter.url .. hopefully that solves this one.

@danro danro closed this as completed Nov 21, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants