Binaryify / OneDark-Pro

Atom's iconic One Dark theme for Visual Studio Code
https://marketplace.visualstudio.com/items?itemName=zhuangtongfa.Material-theme
MIT License
1.52k stars 289 forks source link

VsCode February 2020 (version 1.43) update broke syntax highlighting #380

Closed RadomirPerisic closed 4 years ago

RadomirPerisic commented 4 years ago

https://github.com/microsoft/vscode/issues/92308

I tested on JS only but I read other languages are affected as well.

Temporary fix is to add

"editor.semanticHighlighting.enabled": false

in settings.

With semantic highlighting enabled: image

disabled: image

jacobdcastro commented 4 years ago

Yeah I'm having the same highlighting problem. Hope it gets fixed soon!

Binaryify commented 4 years ago

should wait vscode fix it and release new version, that option shoule be false default

RadomirPerisic commented 4 years ago

https://github.com/microsoft/vscode/issues/92308#issuecomment-596912859

I think theme needs to be updated to support new identifiers.

kamenivskyi commented 4 years ago

issue With imports is the same issue (in React)

Binaryify commented 4 years ago

will adapt that in next version, but need some time, so you guy can set it be false, and watiting next version

jacobdcastro commented 4 years ago

@Binaryify thanks so much for the hard work! Huge fan of the theme.

Binaryify commented 4 years ago

https://github.com/microsoft/vscode/wiki/Semantic-Highlighting-Overview#as-a-theme-author-do-i-need-to-change-my-theme-to-make-it-work-with-semantic-highlighting look like vscode will fix this in 1.43.1

Pandawan commented 4 years ago

Great! So will there be a future update that makes use of this semantic highlighting? Theme works fine without it so imo it’s not necessary, but maybe others think differently?

Binaryify commented 4 years ago

@PandawanFr I think it’s not necessary, and I just try to adapt that and found that will make some breaking change

mrmckeb commented 4 years ago

Does this mean the theme doesn't need changes to support "semantic highlighting"?

Binaryify commented 4 years ago

@mrmckeb I think it doesn't need

mrmckeb commented 4 years ago

This is how highlighting looks in the default Dark theme: image

And the same file in OneDark Pro: image

Maybe it needs a little tweaking?

Binaryify commented 4 years ago

@mrmckeb i don't think it need and the default dark theme use semantic highlighting but other theme will ignore semantic highlighting after vscode update