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

Same colors look dull on VS Code compared to Sublime Text (other applications) #75573

Closed
nksaraf opened this issue Jun 15, 2019 · 10 comments
Closed
Assignees
Labels
electron Issues and items related to Electron font-rendering Font rendering issues upstream Issue identified as 'upstream' component related (exists outside of VS Code)

Comments

@nksaraf
Copy link

nksaraf commented Jun 15, 2019

Issue Type: Bug

Recently tried to switch from Sublime Text to VS Code. Used the same colors as Sublime Text for my token colors and the color were considerably duller in VS Code (than almost all other applications). Tried adding app.commandLine.appendSwitch('force-color-profile', 'srgb') to Mac App bundle Resources/app/out/main.js based on #65816 (comment).

Unsure of additional steps is required to enforce this change in the file, but saving it and restarting VS Code did not have any effect. Also tried logging to console to check if that would work (didn't).

Sublime Text:
Screen Shot 2019-06-15 at 11 12 06 AM

VS Code:
Screen Shot 2019-06-15 at 11 11 42 AM

VS Code version: Code 1.35.1 (c7d83e5, 2019-06-12T14:29:22.216Z)
OS version: Darwin x64 18.6.0

System Info
Item Value
CPUs Intel(R) Core(TM) i5-8259U CPU @ 2.30GHz (8 x 2300)
GPU Status 2d_canvas: enabled
checker_imaging: disabled_off
flash_3d: enabled
flash_stage3d: enabled
flash_stage3d_baseline: enabled
gpu_compositing: enabled
multiple_raster_threads: enabled_on
native_gpu_memory_buffers: enabled
rasterization: enabled
surface_synchronization: enabled_on
video_decode: enabled
webgl: enabled
webgl2: enabled
Load (avg) 1, 1, 1
Memory (System) 8.00GB (0.07GB free)
Process Argv
Screen Reader no
VM 0%
Extensions (2)
Extension Author (truncated) Version
vscode-eslint dba 1.9.0
material-icon-theme PKi 3.8.0

(1 theme extensions excluded)

@nksaraf nksaraf changed the title Same colors look dull on VS Code compared to Sublime Text Same colors look dull on VS Code compared to Sublime Text (other applications) Jun 15, 2019
@Tyriar Tyriar added the font-rendering Font rendering issues label Jun 16, 2019
@bpasero
Copy link
Member

bpasero commented Jun 17, 2019

Can you try to reproduce with our nightly insider builds? You can give our preview releases a try from: https://code.visualstudio.com/insiders/

@bpasero bpasero added the info-needed Issue requires more information from poster label Jun 17, 2019
@vscodebot vscodebot bot removed the new release label Jun 17, 2019
@nksaraf
Copy link
Author

nksaraf commented Jun 17, 2019

I tried the nightly build as well and the results are not much different. Here are the screenshots on the nightly build

VS Code insider
Screen Shot 2019-06-17 at 1 12 27 PM

Sublime Text
Screen Shot 2019-06-17 at 1 11 30 PM

@bpasero bpasero added electron Issues and items related to Electron upstream Issue identified as 'upstream' component related (exists outside of VS Code) and removed info-needed Issue requires more information from poster labels Jun 18, 2019
JakubNalewajek added a commit to JakubNalewajek/vscode that referenced this issue Jul 2, 2019
Chromium team has changed the switch that forces old color rendering in version 62. While this fix is imperfect for reasons described in microsoft#51791 (comment) it does fix issue microsoft#75573
@JakubNalewajek
Copy link

JakubNalewajek commented Jul 2, 2019

Looks like there are multiple ways to force old color rendering:

  1. app.commandLine.appendSwitch('disable-color-correct-rendering');
  2. app.commandLine.appendSwitch('disable-features', 'ColorCorrectRendering');
  3. app.commandLine.appendSwitch('force-color-profile', 'sRGB');

I can confirm that both 2 and 3 work on windows 10 with HDR enabled, 1 (which is in the current build) does not work. I see no difference with HDR disabled.

@nksaraf can you try adding option 2 instead?

@dougal83
Copy link

@JakubNalewajek Thank you for the above info. I was faced with the washed out experience with Win10 & HDR. I followed these instructions to implement (in case this is found by others experience same issue).

@bpasero
Copy link
Member

bpasero commented Aug 5, 2019

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:

@bpasero bpasero added the info-needed Issue requires more information from poster label Aug 5, 2019
@JakubNalewajek
Copy link

On Exploration build, colors are fine on Windows 10 with HDR enabled.

@bpasero bpasero added fixed-in-electron-6.0.x and removed info-needed Issue requires more information from poster labels Aug 7, 2019
@nksaraf
Copy link
Author

nksaraf commented Aug 7, 2019

On exploration build, colors look fine on my Mac as well without requiring any other settings. Also would you suggest I continue to use the exploration build, or is it unstable?

Left - Exploration build
Right - Old build
Screen Shot 2019-08-07 at 2 07 41 AM

@chpxu
Copy link

chpxu commented Aug 7, 2019

@nksaraf the exploration builds are stable enough for me (i use it as my daily driver, even if it is behind insiders sometimes) so I guess as long as you're not wanting much from the next iteration's update, you'll be fine.
Plus it's a great way to stay ahead on the game of electron updates and provide feedback :>

@deepak1556
Copy link
Contributor

Fixed by #81644. It should also be available with next insiders.

@nksaraf
Copy link
Author

nksaraf commented Nov 3, 2019

The problem still persists in Insiders with new version of macos (Top is VS Code, bottom is sublime text 3)
My info from VS Code Insiders:
Version: 1.39.1
Commit: 88f15d1
Date: 2019-10-10T23:35:11.314Z
Electron: 4.2.10
Chrome: 69.0.3497.128
Node.js: 10.11.0
V8: 6.9.427.31-electron.0
OS: Darwin x64 19.0.0
Screen Shot 2019-11-03 at 2 32 36 AM

Sublime Text 3
Screen Shot 2019-11-03 at 2 34 08 AM

@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
electron Issues and items related to Electron font-rendering Font rendering issues upstream Issue identified as 'upstream' component related (exists outside of VS Code)
Projects
None yet
Development

No branches or pull requests

7 participants