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 preview/picker popup in CSS files shows black gradient for color #fe5e5f #62619

Closed
yura-chaikovsky opened this issue Nov 5, 2018 · 3 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug editor-color-picker Editor color picker widget issues insiders-released Patch has been released in VS Code Insiders verified Verification succeeded
Milestone

Comments

@yura-chaikovsky
Copy link

Issue Type: Bug

Color popup shows black gradient for color #fe5e5f, while it is expected to be red.

image

VS Code version: Code 1.27.2 (f46c4c4, 2018-09-12T16:17:45.060Z)
OS version: Windows_NT x64 10.0.17134

System Info
Item Value
CPUs Intel(R) Core(TM) i7-4600U CPU @ 2.10GHz (4 x 2694)
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: disabled_software
rasterization: enabled
video_decode: enabled
video_encode: enabled
webgl: enabled
webgl2: enabled
Memory (System) 7.91GB (1.15GB free)
Process Argv C:\Program Files\Microsoft VS Code\Code.exe
Screen Reader yes
VM 0%
Extensions (4)
Extension Author (truncated) Version
vetur oct 0.13.0
vscode-docker Pet 0.3.1
shader sle 1.1.4
pegs the 0.0.5
@kieferrm kieferrm added the bug Issue identified by VS Code Team member as probable bug label Nov 6, 2018
@DominoPivot
Copy link

I can confirm this is still an issue in the latest version (1.32.2 e64cb27).

@ndngo
Copy link

ndngo commented Apr 12, 2019

There seems to be an issue with how saturatedColor is calculated where sometimes certain values turn black.

For example, the colour #fd5e5f:

fd5e5f-unsaturated
image
This is what the colour looks like before saturation.

After saturation:
fd5e5f-saturated
image

After the saturation the colour turns black, so it displays black in the colour picker.
fd5e5f-colour-picker

This bug also exists even when the colour picker renders the colour properly.
For example, the colour #fe5e4f:
fe5e4f-saturated
image
In this case, the colour is rendered properly because the RGB values seem to be saturated correctly even though the HSV values are black.
fe5e4f-colour-picker

@rebornix rebornix added the editor-color-picker Editor color picker widget issues label Oct 24, 2019
@rebornix rebornix added this to the October 2020 milestone Oct 9, 2020
@rebornix
Copy link
Member

rebornix commented Oct 9, 2020

I think the deeper problem here is our color convert converts hsv(360, 1, 1) to rgb(0, 0, 0) instead of rgb(255, 0, 0).

@rzhao271 rzhao271 added the verified Verification succeeded label Oct 28, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Dec 4, 2020
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 insiders-released Patch has been released in VS Code Insiders verified Verification succeeded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants
@rebornix @yura-chaikovsky @kieferrm @rzhao271 @DominoPivot @ndngo and others