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

Add a setting to support RGB color profile chosen by OS #65816

Closed
vincentmathis opened this issue Dec 29, 2018 · 16 comments
Closed

Add a setting to support RGB color profile chosen by OS #65816

vincentmathis opened this issue Dec 29, 2018 · 16 comments
Labels
feature-request Request for new features or functionality workbench-electron Electron-VS Code issues

Comments

@vincentmathis
Copy link

Version: 1.30.1
Commit: dea8705
Date: 2018-12-18T18:07:32.870Z
Electron: 2.0.12
Chrome: 61.0.3163.100
Node.js: 8.9.3
V8: 6.1.534.41
OS: Linux x64 4.15.0-43-generic

Steps to Reproduce:

  1. take screenshot of vscode and atom (or even gThumb) next to each other
  2. change os/monitor color profile
  3. take another screenshot of both

This is Atom with my default color profile, colors are vibrant
atom-os-profile-thinkpad
This is Atom after I changend my color profile to sRGB in the operating system settings
(Atom itself also has an option to force sRGB which would produce the same result), colors are washed out
atom-os-profile-srgb

This is VSCode with the exact same color theme and hex values (in theory) and default color profile, colors are washed out compared to atom with default color profile
vscode-os-profile-thinkpad
This is VSCode after I changed my color profile to sRGB in the operating system. It looks exactly the same as before.
vscode-os-profile-srgb

Is there a possibility that VSCode would use the OS chosen colorprofile? Currently the colors in VSCode look a little bit washed out on my monitor.

@bpasero bpasero added upstream Issue identified as 'upstream' component related (exists outside of VS Code) electron Issues and items related to Electron labels Dec 31, 2018
@bpasero bpasero changed the title VSCode ignores RGB color profile chosen by OS Add a setting to support RGB color profile chosen by OS Dec 31, 2018
@bpasero bpasero added feature-request Request for new features or functionality workbench-electron Electron-VS Code issues and removed electron Issues and items related to Electron upstream Issue identified as 'upstream' component related (exists outside of VS Code) labels Dec 31, 2018
@bpasero bpasero removed their assignment Dec 31, 2018
@carestad
Copy link

carestad commented Feb 19, 2019

I am seeing this as well, in Ubuntu. I believe it is related to Chrome/Chromium having difficulties detecting screen changes after a suspend/resume cycle, where one might dock/undock a laptop and change monitors (going from a dual monitor setup to just using the built-in for instance).

I reported a bug with chromium a wile back regarding this, but so far the only thing that really works is forcing srgb as a color profile, that way the colors stay the same no matter.

I do believe this only started happening for me in VSCode only a few weeks ago though. Could be due to Electron 3.0 finally being shipped in 1.31?

Also, I can see that Atom allows one to set the color profile in their settings, so why not in VSCode as well?

@mike391
Copy link

mike391 commented Feb 20, 2019

Chrome has been doing this to me as well, but I was able to fix it by forcing sRGB profile in chrome://flags. I cant see this being tough to implement in vscode as well.

@jdsimcoe
Copy link

jdsimcoe commented Feb 21, 2019

I'm seeing this same issues on macOS with the latest build of VSCode Insiders: Version 1.32.0-insider. A manual workaround that seemed to solve it was to add a line of code to main.js inside the VSCode app bundle:

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

@douglasmiranda
Copy link

@felrib
Copy link

felrib commented Jul 4, 2019

The bright colors are gone since last VSCode update (1.36). Using on Ubuntu 19.04.

Trying with --force-color-profile=srgb makes no difference.

@douglasmiranda
Copy link

douglasmiranda commented Jul 4, 2019

Yep! Now it seems to be working properly.

Vscode 1.36
Ubuntu 18.04.2
GeForce GTX 1050 Ti (nvidia-driver-430 open source)


Edit: Now I'm looking at the processes of Vscode and it seems it's launching with --force-color-profile=srgb by default. (At least I haven't changed the launcher)

@felrib
Copy link

felrib commented Jul 4, 2019

Actually I meant that now (1.36) all colors are too opaque. 1.35 was fine.

@douglasmiranda Is it working fine on your setup?

@douglasmiranda
Copy link

Oh I see, I thought you're saying it was ok now.

Here it seems to be ok.

@felrib
Copy link

felrib commented Jul 4, 2019

Thanks for the heads up :)

@codeart1st
Copy link

I have similar issues since 1.36 with a dual monitor setup. If i working on my normal pc monitor without HDR support I see washed out colors. This happens only in combination with a secondary HDR monitor. If I disable the HDR support of the second monitor in Windows all seems to be ok in VS Code.

@AngellusMortis
Copy link

AngellusMortis commented Jul 10, 2019

I have similar issues since 1.36 with a dual monitor setup. If i working on my normal pc monitor without HDR support I see washed out colors. This happens only in combination with a secondary HDR monitor. If I disable the HDR support of the second monitor in Windows all seems to be ok in VS Code.

I have the exact same issue. I cannot upgrade passed 1.35 until this issue is resolved (or you start defaulting to SDR when you have a HDR and SDR monitor). It is a major blocker for me upgrading as I do not think being forced to use the CLI arg to open VS Code all of the time is a proper solution.

@bpasero
Copy link
Member

bpasero commented Aug 5, 2019

People seeing this issue, can please try how the situation is with our new Electron 6 enabled builds:

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

@wangzishi
Copy link

It seems fix this HDR issue on Windows.

image

@felrib
Copy link

felrib commented Aug 5, 2019

image
Left: 1.37.0 Exploration
Right: 1.36.1
Both: using One Dark Pro Vivid theme
Ubuntu 19.04

It's finally resolved!

@bpasero
Copy link
Member

bpasero commented Aug 5, 2019

If this turns out to be fixed with Electron 6, I think we can survive without having this setting configurable.

@deepak1556
Copy link
Contributor

Fixed with #81644

@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 25, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request Request for new features or functionality workbench-electron Electron-VS Code issues
Projects
None yet
Development

No branches or pull requests