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

syntax highlighting #56758

Closed
LexRiver opened this issue Aug 19, 2018 · 8 comments
Closed

syntax highlighting #56758

LexRiver opened this issue Aug 19, 2018 · 8 comments
Assignees
Labels
grammar Syntax highlighting grammar typescript Typescript support issues upstream Issue identified as 'upstream' component related (exists outside of VS Code)

Comments

@LexRiver
Copy link

After the last update if I have a line with comment like this:

export /* comment */ class MyClass{
...
}

then I have a bug with syntax highlightning inside this class.
Previous version of vscode was fine.
It's a typescript *.ts file.

@vscodebot vscodebot bot added the typescript Typescript support issues label Aug 19, 2018
@Nufflee
Copy link

Nufflee commented Aug 19, 2018

A screenshot could be useful here. And can you specify what version were you on when it worked wnad what version are you one now?

@LexRiver
Copy link
Author

Screenshot is below

vscode-bug

Current version:

Version: 1.26.1
Commit: 493869e
Date: 2018-08-16T18:38:57.434Z
Electron: 2.0.5
Chrome: 61.0.3163.100
Node.js: 8.9.3
V8: 6.1.534.41
Architecture: x64

Previous version I don't know, sorry.

@jackhorton
Copy link
Member

This may be related to a bug I just noticed when editing ChakraCore.

https://github.com/Microsoft/ChakraCore/blob/769922256afe021177babc675774b18e9cf54cf2/lib/Runtime/Library/InJavascript/Intl.js#L593-L601

After line 595, the majority of the rest of the document is interpreted as a string. Removing the end-of-line comment does not help. This worked in 1.25.x, but I don't know if it broke with 1.26 or 1.26.1

@jackhorton
Copy link
Member

jackhorton commented Aug 20, 2018

Minimal repro:

image

It specifically fails when concatenating strings across newlines when the trailing + has a space after (note that I have whitespace on -- the trialing space in the failure case is important)

Perhaps I am hitting a different bug -- I can file a new issue for it. Two JS syntax highlighting errors in 1.26.x may be similar root cause, though?

@mjbvz
Copy link
Collaborator

mjbvz commented Aug 20, 2018

@jackhorton Please open a separate issue

@mjbvz mjbvz added the grammar Syntax highlighting grammar label Aug 20, 2018
@mjbvz
Copy link
Collaborator

mjbvz commented Aug 20, 2018

Opened microsoft/TypeScript-TmLanguage#650 to track this upstream

@mjbvz mjbvz closed this as completed Aug 20, 2018
@mjbvz mjbvz added the upstream Issue identified as 'upstream' component related (exists outside of VS Code) label Aug 20, 2018
@jackhorton
Copy link
Member

Is TypeScript-TmLanguage the preferred repo for my issue as well?

@mjbvz
Copy link
Collaborator

mjbvz commented Aug 20, 2018

yes

@vscodebot vscodebot bot locked and limited conversation to collaborators Oct 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
grammar Syntax highlighting grammar typescript Typescript support issues upstream Issue identified as 'upstream' component related (exists outside of VS Code)
Projects
None yet
Development

No branches or pull requests

4 participants