Skip to content
This repository has been archived by the owner on Jul 23, 2023. It is now read-only.

color wheel does not update to brightness #5

Closed
nissaba opened this issue Feb 13, 2018 · 6 comments
Closed

color wheel does not update to brightness #5

nissaba opened this issue Feb 13, 2018 · 6 comments

Comments

@nissaba
Copy link
Collaborator

nissaba commented Feb 13, 2018

The color wheel does not update its colors when the brightness slider is moved.

@EyreFree
Copy link
Member

The brightness slider works well in demo project, can you provide more details?

@nissaba
Copy link
Collaborator Author

nissaba commented Feb 26, 2018

if you move the brightness slider the color picker is not updated, only the current color is.
simulator screen shot - iphone se - 2018-02-26 at 13 22 47
simulator screen shot - iphone se - 2018-02-26 at 13 22 54
simulator screen shot - iphone se - 2018-02-26 at 13 22 58

in the 3 images the color selection disk/circle is alway the same.
normaly in other tools this would update to become fully black if the brightness slider is set to 0
and as the brightness increases the colors in the picker disk would get brighter as it goes up the scale.

as can bee seen in the following images example from XCode color picker.
capture d ecran 2018-02-26 a 13 29 29
capture d ecran 2018-02-26 a 13 29 38
capture d ecran 2018-02-26 a 13 29 44

@nissaba
Copy link
Collaborator Author

nissaba commented Feb 26, 2018

Sorry for the UGE image size.

@nissaba
Copy link
Collaborator Author

nissaba commented Feb 27, 2018

Hi. I finally noticed that in HSB the brightness bar changes as you move the small selector disk. and this is where you know the outcome of the color with adjust brightness will be. The Slider bar is ever so thin it is hard to see and I did not noticed until today that it was chaging as I was moving the selector in the color wheel.

So 1 suggestion is to make that bar thiker, so people that are presbyopic (like me) can notice the change. I did ask my graphic desinger to play with it, he also did not notieced that the sliderbar was chainging color and he as good eyesight.

An other solution is to keep the slider bar on a black to withe gradiant and have the colorwheel update its colormap base on the brightness.

I have updated my fork, as you change the brightness the color map updates.

@EyreFree
Copy link
Member

EyreFree commented Mar 2, 2018

Thanks for your contributions. Your PR has been merged.

You are the first user I know to actually use this library. Are you interested in being a collaborator of this project? In this way, you will be more convenient for the modification and development of this project.

I sent you a invitation, click the link to see: https://github.com/EyreFree/EFColorPicker/invitations.

😀

@nissaba
Copy link
Collaborator Author

nissaba commented Mar 9, 2018

Thanks. I have accepted it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants