-
Notifications
You must be signed in to change notification settings - Fork 30.2k
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
Comments
(Experimental duplicate detection) |
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. |
I'm also seeing this on MacOS 10.14.2 on a 2017 Macbook Pro monitor using the Dark Molokai Theme: Before update (
|
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: |
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... |
@poiru is there anything else required than:
|
@bpasero You need to switch to |
I pushed this change to the exploration build. Can someone verify? |
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 (Is the link in the first post still correct for Linux? Where can I find a more generic landing page for Exploration build?) 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! |
@ablakey should mention, those links are old and you need to update once to get latest! |
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 app.commandLine.appendSwitch("disable-features", "ColorCorrectRendering"); with: app.commandLine.appendSwitch('force-color-profile', 'srgb'); (via 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 |
Just for the record, |
@ablakey's fix worked for me too on Ubuntu 18.10 |
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 |
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. |
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? |
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 |
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 You might have to restart your desktop environment for this to take effect. |
@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. |
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. |
It seems like the only thing blocking the upgrade is #69301. No response since February from Electron team :\ |
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. |
@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 |
@deepak1556 maybe knows |
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. |
Nope, the desaturated colors and artifact problem are both gone in the latest builds with Electron 4 |
I cannot reproduce the issue on latest insiders.
…On Tue, Jun 11, 2019, 12:16 PM Benjamin Pasero ***@***.***> wrote:
Are people 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.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#68001?email_source=notifications&email_token=ACE2T6FDQKYI6FKUYJMVEZTPZ7FXNA5CNFSM4GUX5ZL2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXNVROA#issuecomment-500914360>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ACE2T6EKYJNOWSVV475V43TPZ7FXNANCNFSM4GUX5ZLQ>
.
|
We will ship with Electron 4 with our upcoming release |
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 |
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:
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
The text was updated successfully, but these errors were encountered: