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

Latest version of VSCode breaks City Lights for Babel Language #58

Open
mfix22 opened this issue Jan 15, 2020 · 12 comments
Open

Latest version of VSCode breaks City Lights for Babel Language #58

mfix22 opened this issue Jan 15, 2020 · 12 comments

Comments

@mfix22
Copy link

mfix22 commented Jan 15, 2020

Everything just looks gray and blue now 🤷‍♂

@noudadrichem
Copy link
Collaborator

What is the 'latest version' ? Everything looks fine on my machine.

@mfix22
Copy link
Author

mfix22 commented Jan 16, 2020

I'll close this until I can investigate further. Sorry about the poorly formed issue — was in too big of a rush!

@mfix22 mfix22 closed this as completed Jan 16, 2020
@mfix22
Copy link
Author

mfix22 commented Jan 23, 2020

Screen Shot 2020-01-23 at 2 35 42 PM

Maybe I am wrong, but I thought components are supposed to be rendered as not gray

@mfix22 mfix22 reopened this Jan 23, 2020
@noudadrichem
Copy link
Collaborator

This is true!
Can you provide some more details on versions etc.? I'm still not facing this issue. So a bit more context would be great for me to investigate further.

Thank you in advance!

@mfix22
Copy link
Author

mfix22 commented Jan 28, 2020

Code version: 1.41.1
City lights version: 1.1.5
Babel Javascript (by Michael McDermott) version: 0.0.25

Using macOS, Mojave, 10.14.6

Let me know if there is anything else that would help!

@FezVrasta
Copy link

I can reproduce the same issue, comparing the generated markup/html with the default vscode editor and city lights, I see that the default theme wraps < and the component name in two different span

image

While city lights wraps both in the same:
image

@noudadrichem
Copy link
Collaborator

Hi @mfix22

I think this is due too new updates on how colors are processed...
I'm looking into it at the moment. But for now you can disable semantic highlighting and all should work as the old!

Add "editor.semanticHighlighting.enabled": false to your settings.json

@noudadrichem
Copy link
Collaborator

Hi @FezVrasta,

I'm not sure what you want me to do with a HTML screenshots of VSCode. We don't change any HTML on purpose as the colors are 'just' processed from a JSON file.

@FezVrasta
Copy link

FezVrasta commented Mar 14, 2020

@noudadrichem I don't know, I just noticed this difference and reported back, I have no knowledge about this so I was just trying to be useful.

To be clear, those screenshots are from the vscode dev tools, it's not my own code obviously, it's just the markup generated by vscode to highlight the same segment of code with the two themes.

@FezVrasta
Copy link

editor.semanticHighlighting.enabled is already set to false on my environment anyways.

@mfix22
Copy link
Author

mfix22 commented Mar 14, 2020

Add "editor.semanticHighlighting.enabled": false to your settings.json

@noudadrichem thank you! I will give this a shot 👍

@mfix22
Copy link
Author

mfix22 commented Mar 16, 2020

I am still noticing the issue. The one difference between the two computers I have this theme installed on is that the one using the Babel JavaScript plugin by Michael McDermott is showing the issue above, and the other one isn't, regardless of my editor.semanticHighlighting.enabled setting.

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