-
-
Notifications
You must be signed in to change notification settings - Fork 54
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
Suggestion: Show current color when using color sampler #16
Comments
Currently, this app uses the built-in color sampler which does not have this functionality. To achieve this, I would have to build a completely custom color sampler (which is quite a lot of work) and it would also require "Screen Recording" privacy access in System Preferences. So, yes, it's something I want, but it's also not something I plan to work on anytime soon. |
Tip: You can press Space while sampling a color to show the RGB color values in the picker. It's not configurable though. |
The Swift package DSFColorSampler might be a start in improving the color sampler. It doesn't seem to improve the color information inside the sampler but it does have support for zoom with mouse scroll. |
@W1W1-M I'm aware of that package, but it doesn't do everything I will need to do, so it's easier to make something from scratch. |
Pressing spacebar to see RGB is super helpful. It would be really nice to be able to see the color displayed in the user's preferred format. I had a look through the code, but I'm not a Mac / Swift programmer so I'm not sure where to start 😆 |
I switched from Sip to System Color Picker a few months ago, and I'm very thankful for the great app you built – Thanks!
One thing I'm always missing from Sip, though, is the possibility to see the current color inside the color sampler. It looks like this:
I find this very helpful when I have to pick the color of text, where I'm searching for the darkest color with the color picker, so I don't accidentally select a weaker color of font smoothing. Is there any chance System Color Picker may support this too? Thanks for your consideration.
The text was updated successfully, but these errors were encountered: