Skip to content

Syntax highlighting worse with some color themes after #1337? #1368

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

Closed
J0F3 opened this issue Oct 13, 2020 · 4 comments
Closed

Syntax highlighting worse with some color themes after #1337? #1368

J0F3 opened this issue Oct 13, 2020 · 4 comments
Labels
Resolution-Duplicate Will close automatically.

Comments

@J0F3
Copy link

J0F3 commented Oct 13, 2020

Hi,

In the preview version of the VS Code PowerShell extension the syntax highlighting seems to be worse for many color themes than it is currently in the stable release. I am not quite sure what it is but I assume it was the change in #1337.

Version v2020.6.0:
One Dark Pro:
image
Nord:
image

Version v2020.9.0 (Preview):
One Dark Pro:
image
Nord:
image

Thanks for looking into it!
(I am not sure if I am at the correct place here but I saw the change mentioned above here so I posted it here for now.)

@ghost ghost added the Needs: Triage Maintainer attention needed! label Oct 13, 2020
@rjmholt
Copy link
Contributor

rjmholt commented Oct 13, 2020

@J0F3
Copy link
Author

J0F3 commented Oct 13, 2020

While I not quite understand the cause and what is discussed in #2852 (which is not your problem though 😉) I see that apparently there is already work ongoing regarding this.
So you may close this one if you like.

Thx!

@ghost ghost added the Needs: Maintainer Attention Maintainer attention needed! label Oct 13, 2020
@rjmholt
Copy link
Contributor

rjmholt commented Oct 13, 2020

While I not quite understand the cause and what is discussed in #2852

Essentially the issue is that "accurate token categorisation" and "better color highlighting" aren't actually 100% aligned. Even though we categorise tokens better, because of the way colourschemes work we face issues with the colours ending up looking worse.

I'll close this issue as a duplicate of the other, since we already have some discussions about semantic highlighting.

@rjmholt rjmholt added Resolution-Duplicate Will close automatically. and removed Needs: Maintainer Attention Maintainer attention needed! labels Oct 13, 2020
@rjmholt
Copy link
Contributor

rjmholt commented Oct 13, 2020

Duplicate of PowerShell/vscode-powershell#2852

@J0F3 J0F3 closed this as completed Oct 14, 2020
@TylerLeonhardt TylerLeonhardt removed the Needs: Triage Maintainer attention needed! label Oct 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution-Duplicate Will close automatically.
Projects
None yet
Development

No branches or pull requests

3 participants