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

Electron 3.0.x: Pale/desaturated colors #68001

Closed
atomiks opened this issue Feb 6, 2019 · 33 comments
Closed

Electron 3.0.x: Pale/desaturated colors #68001

atomiks opened this issue Feb 6, 2019 · 33 comments
Assignees
Labels
electron Issues and items related to Electron upstream Issue identified as 'upstream' component related (exists outside of VS Code)
Milestone

Comments

@atomiks
Copy link

atomiks commented Feb 6, 2019

⚠️ bpasero: ⚠️

This will be fixed in Electron 4 when via @poiru electron/electron#15898. Please expect Electron 4 to be out in 2-3 months.

If you cannot wait until then, consider to use our exploration builds that already ship with Electron 4:

We are building exploration builds (you might need to update from within once to get latest) that use a much newer version of our UI framework (Electron version 4.0.x). I wonder if this issue reproduces with one of these builds, could you try? Download:

Original Issue:

  • VSCode Version: 1.31.0
  • OS Version: macOS 10.14.2

Same as atom/atom#17356

The new color profile makes the editor appear desaturated compared to the previous version. Is there an option to use the previous sRGB profile again? (Atom's checkbox doesn't actually work)

Does this issue occur when all extensions are disabled?: Yes

@vscodebot vscodebot bot added the new release label Feb 6, 2019
@vscodebot
Copy link

vscodebot bot commented Feb 6, 2019

(Experimental duplicate detection)
Thanks for submitting this issue. Please also check if it is already covered by an existing one, like:

@jhpratt
Copy link

jhpratt commented Feb 6, 2019

Seconding this on Ubuntu 18.04.

To be clear, this isn't a subtle change. The new colors are actually more vibrant, which is absolutely not what I want. I have a theme with pastel colors, and this change renders some text nearly illegible.

@kika
Copy link

kika commented Feb 7, 2019

Before update (now taken from Android Studio where I have my color scheme and where I copied the colors from):
image

After update:
image

In particular, some shades of green are now indistinguishable.

But before the update, I copied the colors from Android Studio using screen color picker, and now I can obtain the same colors by copying hex values. Which means that at least now VSCode uses the same color matching mechanism.

I'm on MacOS 10.13.6 and use a wide-gamut monitor.

@sam3d
Copy link

sam3d commented Feb 7, 2019

I'm also seeing this on MacOS 10.14.2 on a 2017 Macbook Pro monitor using the Dark Molokai Theme:

Before update (v1.30.2)

screenshot 2019-02-07 at 00 29 13

After update (v1.31.0)

screenshot 2019-02-07 at 00 27 38

It's a kind of subtle change but it's definitely noticeable. Like @jhpratt I have a vibrant colour theme for a reason, I need the high contrast for usability.


Edit: I've simply downgraded to v1.30.2 and will keep it there until this is fixed.

@tannergaucher
Copy link

Also getting this after upgrade on MacOS 10.14.2 and my 2017 Macbook Pro with Atom One Dark theme. Colors are considerably less saturated.

screen shot 2019-02-07 at 11 50 35 am

@bpasero bpasero added upstream Issue identified as 'upstream' component related (exists outside of VS Code) electron Issues and items related to Electron electron-3.0.x-update labels Feb 7, 2019
@bpasero
Copy link
Member

bpasero commented Feb 7, 2019

This will be fixed in Electron 4 via @poiru electron/electron#15898. Please expect Electron 4 to be released in stable in 2-3 months.

If you cannot wait until then, consider to use our exploration builds that already ship with Electron 4:

We are building exploration builds that use a much newer version of our UI framework (Electron version 4.0.x). I wonder if this issue reproduces with one of these builds, could you try? Download:

@bpasero bpasero changed the title Pale/desaturated colors in 1.31.0 Electron 3.0.x: Pale/desaturated colors Feb 7, 2019
@atomiks
Copy link
Author

atomiks commented Feb 7, 2019

The darwin exploration build still has the desaturated colors. I'm also noticing some color artifacts in the editor background on 1.31 on the right side in the form "tiles" around where the line selection is, which appear to be fixed in the darwin build. Does there need to be a checkbox to use the old color profile?

For anyone wanting the vibrant colors back, on macOS, you can set the whole OS to use sRGB in System Prefs > Displays > Colour:

This has the side effect of making everything in the OS appear overly vibrant/saturated, something that I only like in my editor for the bright syntax highlighting...

@bpasero
Copy link
Member

bpasero commented Feb 7, 2019

@poiru is there anything else required than:

app.commandLine.appendSwitch('disable-features', 'ColorCorrectRendering');

@poiru
Copy link

poiru commented Feb 7, 2019

@bpasero You need to switch to app.commandLine.appendSwitch('disable-color-correct-rendering').

@bpasero
Copy link
Member

bpasero commented Feb 7, 2019

I pushed this change to the exploration build. Can someone verify?

@ablakey
Copy link

ablakey commented Feb 7, 2019

Not 100% sure I'm dealing with the same issue, but it looks unchanged. Downloaded Exploration build 1 minute ago.

Version: 1.31.0-exploration
Commit: 37818ac

(Is the link in the first post still correct for Linux? Where can I find a more generic landing page for Exploration build?)

image

Just wanted to share that this is a blocking issue for me. Call me a broken human but this faded colour is more distracting to my job than a fire alarm going off. I eagerly hope we can help get this fixed in any build. Otherwise I'll seek info on how to revert to 1.30 for the next 2-3 months. Thanks for your help!

@bpasero
Copy link
Member

bpasero commented Feb 7, 2019

@ablakey should mention, those links are old and you need to update once to get latest!

@ablakey
Copy link

ablakey commented Feb 7, 2019

Derp. Of course!

image

You might have to take my word for it. It is noticeably better to my eyes. The background is distinctly darker.

@ablakey
Copy link

ablakey commented Feb 8, 2019

So I did something unconscionably awful.

I didn't want to use a different version of code because I didn't want to migrate all of my settings, extensions, etc. over.

In /usr/share/code/resources/app/out/main.js I replaced:

app.commandLine.appendSwitch("disable-features", "ColorCorrectRendering");

with:

app.commandLine.appendSwitch('force-color-profile', 'srgb');

(via sed or your favourite editor with sudo privileges)

This probably relies heavily on my computer's environment settings, the monitor's profile, and whatnot. So I anticipate this is not a globally useful solution.

This modifies built code in /usr/share/ which is preeeeetty awful. But it works. I expect it will get clobbered when vscode gets updated next. I can't understate how "wrong" this is. But it solved my problem very easily and may help others.

@poiru
Copy link

poiru commented Feb 8, 2019

Just for the record, --force-color-profile=srgb does not work on macOS.

@Zeex
Copy link

Zeex commented Feb 9, 2019

@ablakey's fix worked for me too on Ubuntu 18.10

@dhowden
Copy link

dhowden commented Feb 25, 2019

Just tried the exploration build for macOS and this does not appeared to be fixed there. Is there a more up-to-date exploration build we should be trying?

I have tried build 37818ac3597b4afda264b05226b0b77fb8e85f41.

@ablakey
Copy link

ablakey commented Mar 18, 2019

Are my eyes going bad or does this appear to be fixed in the latest release? Either that, or this issue isn't replicated in Bionic, which I switched to recently.

@dhowden
Copy link

dhowden commented Mar 18, 2019

I've been switching back and forth between the recent versions (back to 1.30.2). I still see the same issue.

Which version are you trying?

@atomiks
Copy link
Author

atomiks commented Apr 2, 2019

I can confirm it's fixed in the macOS download link version (after updating it) and the colors are vibrant again 🌈. When does that version go stable? I've been staying on 1.30.2

@nchengyeeshen
Copy link

I'm on Fedora 29, VSCode 1.33.0 and colours are still weird unless I force the colour profile to become sRGB.

Another temporary fix I've found is to override the .desktop file by copying it to ~/.local/share/applications and then adding --force-color-profile=srgb at the Exec line in code.desktop, like so
Exec=/usr/share/code/code --force-color-profile=srgb --unity-launch %F.

You might have to restart your desktop environment for this to take effect.

@jhpratt
Copy link

jhpratt commented Apr 11, 2019

This will be fixed in Electron 4 via @poiru electron/electron#15898. Please expect Electron 4 to be released in stable in 2-3 months.

@bpasero Any updates, or an ETA on when Electron 4 will ship beneath Code? Every single update forces me to pull this issue back up to find the necessary file and flag I need to switch. At the very least, putting in an option to toggle it would be great.

@jhpratt
Copy link

jhpratt commented Apr 18, 2019

@bpasero

I'm interpreting a lack of response (given your activeness on GitHub) as "we have no timeframe", which is a shame, really. Tons of people are affected by this, and there's no effort to change it.

Upgrading to Electron 4 is the suggested solution, but this has been deferred past the April release (due to come out May 6), per #71830. Your estimated timeframe was 2-3 months, and the release on May 6 is the long end of that. I (along with others) would very much appreciate an updated estimate.

At the absolute least, adding a dropdown in the settings to toggle the rendering would solve the issue temporarily.

@atomiks
Copy link
Author

atomiks commented Apr 18, 2019

It seems like the only thing blocking the upgrade is #69301. No response since February from Electron team :\

@bpasero
Copy link
Member

bpasero commented Apr 18, 2019

@jhpratt I have just triggered a new Exploration build with latest changes. Have you considered staying on that version until we did the move?

Download:

@bpasero
Copy link
Member

bpasero commented Apr 18, 2019

As for the setting: since this is something that needs to be set very early on on startup, we do not have access to our settings file yet. Since startup performance is critical to us, we (so far) have stayed away from blocking the startup to read the settings file.

@dhowden
Copy link

dhowden commented May 22, 2019

@bpasero Has the latest exploration build broken this again? This was working fine, seems to have stopped working for me today. I updated to macOS 10.14.5 last night, so that might have caused the problem (hope not, less likely to be fixable!).

Edit: I'm running Version 1.35.0-exploration (1.35.0-exploration) (1bdebe7e66e42d73b62c578b46fdc9a3e84c99d5)

@bpasero
Copy link
Member

bpasero commented May 22, 2019

@deepak1556 maybe knows

@bpasero
Copy link
Member

bpasero commented Jun 11, 2019

Are people (@atomiks) still seeing this with latest VSCode insider builds? You can give our preview releases a try from: https://code.visualstudio.com/insiders/

Those have been updated to Electron 4. Just curious.

@atomiks
Copy link
Author

atomiks commented Jun 12, 2019

Nope, the desaturated colors and artifact problem are both gone in the latest builds with Electron 4

@ablakey
Copy link

ablakey commented Jun 12, 2019 via email

@bpasero
Copy link
Member

bpasero commented Jul 1, 2019

We will ship with Electron 4 with our upcoming release 1.36 this week. As such I am closing this issue. To benefit from the update already today, consider to use our insiders version: https://code.visualstudio.com/insiders/

@bpasero bpasero closed this as completed Jul 1, 2019
@bpasero bpasero added this to the June 2019 milestone Jul 1, 2019
@dhowden
Copy link

dhowden commented Jul 16, 2019

Running the latest insiders, looks like the color saturation issue has returned... Reporting here as it is related, apologies if this should be a new issue!

Version: 1.37.0-insider
Commit: 894056d
Date: 2019-07-15T05:24:02.940Z
Electron: 4.2.5
Chrome: 69.0.3497.128
Node.js: 10.11.0
V8: 6.9.427.31-electron.0
OS: Darwin x64 18.6.0

@vscodebot vscodebot bot locked and limited conversation to collaborators Aug 15, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
electron Issues and items related to Electron upstream Issue identified as 'upstream' component related (exists outside of VS Code)
Projects
None yet
Development

No branches or pull requests