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

Type and variable names are the same colour #19

Open
spapaseit opened this issue Feb 19, 2023 · 2 comments
Open

Type and variable names are the same colour #19

spapaseit opened this issue Feb 19, 2023 · 2 comments
Labels
enhancement New feature or request

Comments

@spapaseit
Copy link

Let me start off by saying thank you for your hard work on this incredible theme, it keeps getting better and better!

I just wanted to add a request: To differentiate the colour of Type and Variable names, like in the VS Code version of the Darcula theme.

Here is a side by side of the same file in VS2022 and VS Code:
image

As you can see, the colour differentiation makes it easier to see what's what at a glance.

Thanks again

@FINNSEEFLY
Copy link
Owner

Hi

Initially, I tried to make the theme as similar as possible to Darcula from Rider.

Darcula theme in Rider 2022.3.2

image

I fully understand the wish to add some kind of highlighting for classes.
In my daily work, I use a slightly modified Rider Dark theme (I made interfaces different from classes).

Rider Dark theme (modified) in Rider 2022.3.2

image

I also like the highlighting offered in Fleet.

Fleet

image

The last two examples differ from each other not only in colors, but also in the presence or absence of their own colors, for example, for private class fields.
However, speaking specifically about Darcula, it looks like some feature of this theme. This theme came from IntelliJ IDEA, where it has been in this form for years and many Java programmers have used it in this form. Thus, if I change this, in our opinion, inconvenient feature, it may be an unpleasant surprise for those who are already used to this theme and were looking for it in this form in order to switch to C# without unnecessary discomfort.
At the same time, I understand that this feature does not contribute to productive work.

I will definitely come back to this when I start working on the next version. I also think it's a good option to just add the version with the changed highlighting to the extension as a separate theme, and leave the old one as it is.

I would be grateful if you could share a link to the Darcula version you mentioned.

@FINNSEEFLY FINNSEEFLY added the enhancement New feature or request label Feb 19, 2023
@spapaseit
Copy link
Author

Thanks for the detailed explanation, I totally understand the motivation.

Interestingly, rokoroku decided to modify the theme when he created the VS Code version of the Darcula theme "to match look and feel with VS Code IDE.", as he states in the README.

I also think it's a good option to just add the version with the changed highlighting to the extension as a separate theme

I like the idea :)

Thanks yet again for the time and effort you spend on this theme!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants