Change the preset color

[mmahmoudian]

Feature Description

At the moment the hexadecimal number of the colors cannot be modified and instead the user have to add a new color and delete the one they don't like. It would be nice to be able to modify them on the spot.

Also would be nice to be able to re-arrange the colors in GUI

[deo002]

@mmahmoudian How about removing the spinbox altogether? Users can select a preset by clicking it. In the spinbox's place, we can add 2 arrow buttons for reordering.

[mmahmoudian]

I don't know which one of the two circles are you referring to as "spinbox", but I assume it is the lower one. For the sake of this comment and until you correct me I'm going to call the top one "colorwheel" and the lower one "spinbox".

I believe the most intuitive UI would be that the spinbox goes in the center of the colorwheel and spinbox is only shown when one of the circles in the colorwheel is selected. And for reordering, I think dragging the circles in the preset colors is the intuitive way. In general something similar to what @veracioux proposed before

All these said, I don't know how easy or hard it is to implement these, plus this is just my personal opinion and other should also express their opinions.

But for the time being we need a method to modify the color in an existing preset color circle.

[deo002]

I don't know which one of the two circles are you referring to as "spinbox"

The widget used for selecting colors.

[deo002]

And for reordering, I think dragging the circles in the preset colors is the intuitive way.

I like this idea though. Better than the 2 arrow one.

[mmahmoudian]

The widget used for selecting colors.

Imho this is very unintuitive. As a user I expected to be able to edit the text of that box you showed to change the hex of the selected color. I think the circles should be clickable and selectable (instead of selecting through the box you showed) and then instead of this box there should be a an input field to modify the hex of the selected circle.

[deo002]

I think the circles should be clickable and selectable (instead of selecting through the box you showed) and then instead of this box there should be a an input field to modify the hex of the selected circle.

Sounds good! I would like to work on it.

[mmahmoudian]

@deo002 let's first hear the opinion of other devs before diving into the code.

@borgmanJeremy @veracioux what do you think? Any feedbacks on the thread above?

[borgmanJeremy]

I think what was discussed here sounds good to me. Dragging the circles seems like it might be complicated but it seems like the most intuitive way to do it.