atom / language-yaml

YAML package for Atom
Other
24 stars 39 forks source link

Broken syntax highlight #8

Closed ts33kr closed 10 years ago

ts33kr commented 10 years ago

image

kevinsawicki commented 10 years ago

Could you include the raw text in a comment or the issue body?

ts33kr commented 10 years ago

Sure thing, Kevin. Here is the link to the mentioned file https://github.com/ts33kr/granite/blob/master/locale/tracked.yaml

ts33kr commented 10 years ago

Basically, it looks like there are two different bugs: a) for some reason $ is tokenized as some sort of token, which it is certainly not; b) the ' and " are always recognized as string delimiters, which is wrong by the spec. A couple of parsers and the JetBrains IDEs seems to pick up this contents just fine.

kevinsawicki commented 10 years ago

Thanks so much for reporting this, a fix for it will be in the next Atom release, 0.121.0

ts33kr commented 10 years ago

Kevin, thanks for the swift reaction. :)