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

macOS Sierra: background artifacts #12473

Closed
maxence-charriere opened this issue Sep 22, 2016 · 54 comments
Closed

macOS Sierra: background artifacts #12473

maxence-charriere opened this issue Sep 22, 2016 · 54 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug electron Issues and items related to Electron upstream Issue identified as 'upstream' component related (exists outside of VS Code) verified Verification succeeded
Milestone

Comments

@maxence-charriere
Copy link

  • VSCode Version: Code 1.5.2 (66f37fd, 2016-09-12T13:05:12.065Z)
  • OS Version: Darwin x64 16.0.0

Steps to Reproduce:

  1. just launch Visual studio on macOS Sierra
@maxence-charriere
Copy link
Author

screen shot 2016-09-22 at 20 13 57

@bpasero
Copy link
Member

bpasero commented Sep 22, 2016

@maxence-charriere can you try if this reproduces with our insider build too?
You can give our preview releases a try from: http://code.visualstudio.com/Download#insiders

@maxence-charriere
Copy link
Author

screen shot 2016-09-22 at 20 14 58

@maxence-charriere
Copy link
Author

ok this one on Version 1.6.0-insider (1.6.0-insider):
screen shot 2016-09-22 at 20 21 26

@chrmarti chrmarti added the bug Issue identified by VS Code Team member as probable bug label Sep 22, 2016
@jaysylvester
Copy link

I'm seeing this as well.

Not sure if this helps, but I'm also seeing a similar problem in Chrome, which I submitted to them also. Screenshot below is Daring Fireball in Safari vs. Chrome:

safari-vs-chrome

@mrmlnc
Copy link
Contributor

mrmlnc commented Sep 23, 2016

@bpasero bpasero added the upstream Issue identified as 'upstream' component related (exists outside of VS Code) label Sep 23, 2016
@bpasero bpasero added this to the Backlog milestone Sep 23, 2016
@bpasero
Copy link
Member

bpasero commented Sep 23, 2016

Thanks for linking the related issues here. The related chrome bug is: https://bugs.chromium.org/p/chromium/issues/detail?id=633805

@bpasero
Copy link
Member

bpasero commented Sep 23, 2016

@maxence-charriere @jaysylvester that issue talks about a possible fix using --force-gpu-rasterization, did you try that? maybe pass it to the starting of code?

@jaysylvester
Copy link

jaysylvester commented Sep 23, 2016

@bpasero Just tried --force-gpu-rasterization with VS Code and that fixes it.

@bpasero
Copy link
Member

bpasero commented Sep 23, 2016

@jaysylvester great to hear that 👍

@bpasero bpasero changed the title Problem with ui colors on macOS sierra macOS Sierra: background artifacts Sep 23, 2016
@maxence-charriere
Copy link
Author

how to use it? I tried code --force-gpu-rasterization on the console but nothing changed.

@bpasero
Copy link
Member

bpasero commented Sep 23, 2016

yeah, from the command line when launching an instance.

@l2dy
Copy link

l2dy commented Sep 23, 2016

/Applications/Visual\ Studio\ Code.app/Contents/MacOS/Electron --force-gpu-rasterization

@FabianLauer
Copy link

Since 1.6.0, I'm seeing some artifacts even with --force-gpu-rasterization.
Also, when setting "window.zoomLevel" to any value other than 0 the artifacts disappear, regardless of --force-gpu-rasterization.

@xgt001
Copy link

xgt001 commented Oct 18, 2016

@FabianLauer Thanks, that did the trick for me!

@bpasero
Copy link
Member

bpasero commented Apr 10, 2017

Todays VS Code insider build comes with Electron 1.6.x, would be interesting to hear if this update solves this issue for anyone: http://code.visualstudio.com/Download#insiders

@Codelica
Copy link

Thanks for the heads up!

Insider build fixed it for me without any workarounds. 👍

(Using a calibrated i1 color profile)

@jfieber
Copy link

jfieber commented Apr 10, 2017

It does. The colors are substantially different than before. Side by side with the same theme, the insider version colors are way more saturated.

@bpasero
Copy link
Member

bpasero commented Apr 11, 2017

Awesome 👍

@bpasero bpasero modified the milestones: April 2017, Backlog Apr 11, 2017
@biscoffCity
Copy link

didn't work. Had to set"window.zoomLevel": 0.001 and I still see screen flickers.

@bpasero
Copy link
Member

bpasero commented Apr 17, 2017

@am6377 are you on our insiders build?

@biscoffCity
Copy link

biscoffCity commented Apr 17, 2017 via email

@bpasero bpasero modified the milestones: Backlog, April 2017 Apr 17, 2017
@bpasero
Copy link
Member

bpasero commented Apr 17, 2017

So it looks like the Electron update is only resolving it for some people.

@epogue
Copy link

epogue commented Apr 17, 2017

@bpasero The Insiders update fixed the issue for me. No need for the zoomLevel hack now.

@biscoffCity
Copy link

@bpasero I tried #insider build but I can't use custom or other built-in display profile except for the standard color LCD which actually doesn't look really nice IRL. The only way it works for me is setting "window.zoomLevel": 0.001 and even with that I get lots of random screen flickers.

screen shot 2017-04-17 at 4 39 48 pm

screen shot 2017-04-17 at 4 40 02 pm

@jfieber
Copy link

jfieber commented Apr 17, 2017

With the current insiders build, using that specific profile (ITU-R BT.709-6), and the default dark theme, I'm not seeing the behavior, either on the built in display (5k iMac) or external display. With the current non-insiders build, I initially thought the window.zoomLevel hack worked, but after using it for awhile, it was clear that it helped some, but didn't eliminate the issue.

@bpasero
Copy link
Member

bpasero commented Apr 26, 2017

The issue is resolved for me, I just verified that. No clue why some people are still seeing it though when trying our insiders build.

@Codelica
Copy link

It fixed all 3 machines I use which have custom profiles. Purely a guess, but maybe some people are downloading the normal version from the link posted (https://code.visualstudio.com/Download#insiders), not realizing the Insiders build is a text link at the bottom.

@bpasero
Copy link
Member

bpasero commented Apr 26, 2017

Yeah, sounds like it.

@bpasero bpasero closed this as completed Apr 26, 2017
@bpasero bpasero modified the milestones: April 2017, Backlog Apr 26, 2017
@biscoffCity
Copy link

I would like to apologize to @bpasero and I would like to thank @Codelica because I realized I didn't download the insiders build. I thought the link with #insiders took to me to the insiders built. I didn't scroll down. The direct link to insiders is
(https://code.visualstudio.com/insiders) WITHOUT the #
My fault :)

@bpasero bpasero added the bug Issue identified by VS Code Team member as probable bug label Apr 27, 2017
@roblourens roblourens added the verified Verification succeeded label Apr 28, 2017
@will-stone
Copy link

Came across this issue when scratching my head that the background colour I was setting in my theme was showing differently on the screen (i.e. when checking it with the Digital Color Meter). I did download the Insiders build and I can confirm it's fixed there for me. Can I ask when this fix will roll out? Theming is feeling rather futile while my colours don't match :-(

@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug electron Issues and items related to Electron upstream Issue identified as 'upstream' component related (exists outside of VS Code) verified Verification succeeded
Projects
None yet
Development

No branches or pull requests