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

CoffeeScript does not recognize #{} #1678

Closed
Defman21 opened this issue Jun 7, 2016 · 5 comments
Closed

CoffeeScript does not recognize #{} #1678

Defman21 opened this issue Jun 7, 2016 · 5 comments
Assignees
Milestone

Comments

@Defman21
Copy link
Contributor

Defman21 commented Jun 7, 2016

Short Summary

variable = 1
console.log "test #{variable}"

Komodo does not highlight variable inside a string as identifier, so #{} does nothing there.

console.log "hello #{["test", ["123"]}"

Komodo highlights ["test", ["123"] not properly:

Imgur

@mitchell-as
Copy link
Contributor

Thanks. Scintilla's CoffeeScript lexer doesn't support string interpolation yet.

@mitchell-as mitchell-as added this to the Perpetual milestone Jun 7, 2016
@mitchell-as mitchell-as self-assigned this Jun 7, 2016
@Naatan
Copy link
Member

Naatan commented Jun 7, 2016

Note this works for Ruby, which has veeery similar syntax.

@Defman21
Copy link
Contributor Author

Defman21 commented Jun 7, 2016

I sure CoffeeScript supports multi-line strings, so #1677 :)

@Defman21
Copy link
Contributor Author

Thank you Mitchell!

@mitchell-as
Copy link
Contributor

Note: Scintilla patch submitted upstream.

Naatan pushed a commit that referenced this issue Nov 8, 2016
#1678

Patch submitted upstream.

rn=

(integrated from the KomodoIDE master branch change a786c65 by Mitchell <mitchellb@activestate.com>)

Komodo/KomodoIDE@a786c65
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants