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

Color picker: Broken pixel along vertical axis #32470

Closed
michelkaporin opened this issue Aug 14, 2017 · 9 comments
Closed

Color picker: Broken pixel along vertical axis #32470

michelkaporin opened this issue Aug 14, 2017 · 9 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug editor-color-picker Editor color picker widget issues
Milestone

Comments

@michelkaporin
Copy link
Contributor

Version: Insiders b73b7df

image

See the screenshot above. Interestingly, not all color values produce this result (e.g. #68c3a3 paints fine).

@joaomoreno
Copy link
Member

He's alive!!! Do you like the new horizontal picker bars? 🥇

Funny broken pixel, though. Does it always reproduce on your machine? No steps, just open it?

@joaomoreno joaomoreno added the info-needed Issue requires more information from poster label Aug 14, 2017
@michelkaporin
Copy link
Contributor Author

michelkaporin commented Aug 14, 2017

Haha, I am here! Yeah, they're awesome, like them 👍 Didn't have a time for a final touch on them before leaving, was focused on finishing off the core implementation 😄

It always reproduces, no steps, just pick the #333333 color.

Also I've looked at your code refactorings around the picker, you made it really sleek 👍 nice work! 🔢

@joaomoreno
Copy link
Member

Strange:

image

@joaomoreno joaomoreno added this to the Backlog milestone Aug 14, 2017
@joaomoreno joaomoreno added bug Issue identified by VS Code Team member as probable bug editor-color-picker Editor color picker widget issues and removed info-needed Issue requires more information from poster labels Aug 14, 2017
@joaomoreno joaomoreno assigned rebornix and unassigned joaomoreno Aug 14, 2017
@michelkaporin
Copy link
Contributor Author

@joaomoreno really strange, and the code remained the same as I've seen.

If I tweak window.zoomLevel to values different to default, it starts to appear fine.

OS: Windows
Resolution: 1920x1080
Scaling: 100%

@rebornix
Copy link
Member

It took me a while to find the way to reproduce (before that I even wanted to ask you to check whether your screen is broken for a second ...).

  • No reproduce on macOS
  • Reproduce on Windows 10, Scaling 100% and Zoom Level 0. If we change the Scaling or Zoom Level, the green line is gone.
  • Chrome Dev Tools doesn't have this problem.

@rebornix
Copy link
Member

After exposing the API to monaco, I can reproduce this issue in Chrome on macOS always with no scale. Once I scale the webpage, the green line is one.

image

This is probably a blocker for releasing to Monaco Editor @joaomoreno . @alexandrudima any idea of how this magic green dots happen in canvas?

rebornix added a commit that referenced this issue Aug 23, 2017
@rebornix
Copy link
Member

It only happens in Chrome (macOS 100% Zoom and Windows 10 Hdpi 100% Scale), didn't see any issue in Safari and Edge. Tried several ways to troubleshoot but didn't see what we did wrong. A workaround here is adding a middle stop point to the linear gradient, which makes Chromium canvas rendering correct.

@joaomoreno
Copy link
Member

Their constant position might give us a clue on why they happen... although I got none. 😕

@rebornix
Copy link
Member

The workaround seems good, so visually this issue is gone. I'd like to put it into backlog and one day I can know the root cause of this canvas problem.

@vscodebot vscodebot bot locked and limited conversation to collaborators Jan 13, 2018
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 editor-color-picker Editor color picker widget issues
Projects
None yet
Development

No branches or pull requests

3 participants