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

[Colour accuracy issue] doom-dark+ (vscode) theme too vibrant? #427

Open
ianyepan opened this issue Mar 21, 2020 · 9 comments
Open

[Colour accuracy issue] doom-dark+ (vscode) theme too vibrant? #427

ianyepan opened this issue Mar 21, 2020 · 9 comments
Labels
is:bug Something isn't working as intended theme:dark+
Milestone

Comments

@ianyepan
Copy link
Contributor

In the screenshot below (Visual Studio Code left, Emacs right), we can see that the colours in Emacs are more vibrant than those in Visual Studio Code (use a colour-hex-detector if you find the differences too small), despite supposedly using the same theme.
The vibrancy includes the green comments, the blue keywords and yellow functions etc. As if there has been a vibrancy filter that was masked over my Emacs interface and the contrast was tuned higher.
@ema2159 Hi Emmanuel! I believe you were the original author of this port -- how did you fetch the official colours from VSCode?
On second thought, it could potentially be the problem in how an 'electron' app renders the same colours though -- in which case it's not an issue of doom-themes at all.

Screenshot 2020-03-21 at 23 45 26

@ema2159
Copy link
Contributor

ema2159 commented Mar 23, 2020

That's a pretty good question. I would need to look further into it. As I remember, I got the colors from here: https://github.com/microsoft/vscode/blob/master/extensions/theme-defaults/themes/dark_plus.json
Not only that file but some others inside that repo.
All the colors seem right, except for the comments that seem a little bit off. I'll try to check what's going on

@ianyepan
Copy link
Contributor Author

Thanks! For your reference, I'm on macOS Catalina using the default Colour LCD display profile.

@ianyepan
Copy link
Contributor Author

@ema2159 I came across the thread, might be of help for those interested: microsoft/vscode#65816

@ema2159
Copy link
Contributor

ema2159 commented Mar 27, 2020

@ianpan870102 interesting, so it was Windows's VS Code fault all along?

@ianyepan
Copy link
Contributor Author

The colour rendering issue has been found on macOS, Windows, and Linux platforms -- I think you meant "Microsoft's" VSCode fault right? That'd be my best guess too, since I double checked the colours defined in VSCode's theme and they are indeed not the right colours if I detect it system-wide. This doom-theme port has chosen the official colours of VSCode.
My port of vscode-emacs theme (https://github.com/ianpan870102/vscode-dark-plus-emacs-theme) took the approach of hand-picking every colour using a colour picker directly from the UI, so you could say that it "looks" the most like VSCode, but it won't be the official colours -- since VSCode can't display their official colours either.
Hope this makes sense.

@ianyepan
Copy link
Contributor Author

To conclude, the doom-theme "too vibrant" issue is actually how VSCode should look like.

@ema2159
Copy link
Contributor

ema2159 commented Mar 30, 2020

@ianpan870102 that's cool. So it seems VS Code has problems rendering colors properly who would've thought. Should we close the issue then?

@ianyepan
Copy link
Contributor Author

Sure, I'll close this one.

@ianyepan ianyepan reopened this May 19, 2020
@ianyepan
Copy link
Contributor Author

ianyepan commented May 19, 2020

@ema2159 Hi there! So the other day I was having a look in Microsoft's vscode github repo and extracted these colours:

Official colors from Microsoft's Visual Studio Code:
"#DCDCAA"
"#4EC9B0"
"#C586C0"
"#9CDCFE"
"#51B6C4"
"#CE9178"
"#d16969"
"#d7ba7d"
"#569cd6"
"#C8C8C8"
"#d4d4d4"
"#b5cea8"
"#f44747"
"#6A9955"

It seems that they have "updated" their color scheme, because the red and magenta are consistent with the ones in doom-dark+, but the others (blue, teal, yellow, etc.) are not.

@hlissner hlissner added is:bug Something isn't working as intended confirmed theme:dark+ labels Sep 16, 2021
@hlissner hlissner added this to the Backlog milestone May 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is:bug Something isn't working as intended theme:dark+
Projects
None yet
Development

No branches or pull requests

3 participants