-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Components: Support the EyeDropper
API in ColorPicker
#34487
Comments
EyeDropper
APIEyeDropper
API to the ColorPicker
EyeDropper
API to the ColorPicker
EyeDropper
API in ColorPicker
Here's a proposal for the eyedropper:
|
cc'ing @mtias @pablohoneyhoney and @jasmussen regarding the potential UX around an eyedropper tool in the `ColorPicker. (example codepen: https://codepen.io/matanbobi/pen/GREoZvo) |
Nice! For some reason I'm not able to see the codepen Not sure if we want this right away and that's why it wasn't explored initially. If we do, there are cases where the color picker doesn't have a header or it isn't clear if we do need that header all times. It might work with the RGBA codes but perhaps the eyedropper could live alongside the |
It does unbalance the UI heavily, no? |
I'll think about alternative ways to display the color, but I think we could pull that off by just slightly increasing the width of the component and reducing the space between the color parts: Quick test: Question: the blue color implies that the RGBA are links, right? What happens when the user clicks them? |
I am able to run the codepen on the latest stable Chrome for MacOS ( |
I'm also on that version, but it didn't work for me I had to install Chrome Canary. |
This is a sweet feature. Because it's such an advanced browser feature that it's barely rolled out to Chrome yet, though, I would echo Pablo that it seems worth it to restrict the button to exist in context of other advanced configuration tools for the time being. Then as browser support becomes wider, the button could literally graduate in the UI. That said, it could be nice to include a small enhancement for power-users, such as allowing people to press the i key to invoke the eye-dropper when the color picker is open. It seems important to change the cursor also, though, when the tool is selected. Ideally we create an entirely custom cursor, pending browsers adding a system-level eye-dropper cursor. Altenately, the crosshair cursor might hold us over. |
Should we implement this feature now (with a check and if not supported we don't render anything)? Or it is something for the future when browser support increases? |
Personally, I think this would a nice-to-have feature with (at the moment) a low priority. We could probably use our resources for higher-priority issue and come back to this once browser support looks a bit better. |
Why
Adding an Eye Dropper tool to the
ColorPicker
would enhance its UX and features setWhat
We should embed an Eye Dropper tool following the
EyeDropper
APIs:Note: this API are not stable yet and, at the time of writing, is only supported in Chrome >= 95
A/C
ColorPicker
)ColorPicker
is dimissedThe text was updated successfully, but these errors were encountered: