[FR] More colours or colour picker for select

[OctoShotG]

Description

More colour options or a colour picker, such as brown, white, black, red, etc

Impact

Easier to make statuses more distinct through colours visible in grid view

Additional Context

No response

[MayurSMahajan]

Should more colours be added here, or a way to add a custom colour should be added ?

[OctoShotG]

Being able to pick any custom colour with something such as a RGB value would be be great for customising- most of the colours now feel quite dark and don't fit the best with some themes.

[nebiyuelias1]

So is the custom color input the way to go? I will be happy to work on this issue @apomalyn.

[MayurSMahajan]

Hey @nebiyuelias1, assigning this to you. Just make sure that the custom colour input is consistent with custom colour inputs we have used in other places within appflowy. Have fun contributing!

[nebiyuelias1]

Hey @nebiyuelias1, assigning this to you. Just make sure that the custom colour input is consistent with custom colour inputs we have used in other places within appflowy. Have fun contributing!

Oh exciting! So I saw some color picker with a dialog in the settings page. Do you think I should use that one?

[MayurSMahajan]

Something more like this. This appears when you try changing highlight or text colour

[nebiyuelias1]

@MayurSMahajan I was able to add the custom color picker to the UI, but I was wondering how I could modify the protocol buffer to allow custom input. Right now it seems it's set to use integer values with an enum. Do you have any suggestions?

[MayurSMahajan]

@nebiyuelias1 Actually I am not the best person for this question, I haven't really worked on the colour pickers. I would advice you look up the existing implementation in here:
https://github.com/AppFlowy-IO/appflowy-editor/blob/main/lib/src/editor/toolbar/desktop/items/color/color_picker.dart

See if that helps, otherwise we'll ask someone who knows this for help.

[nebiyuelias1]

@MayurSMahajan Thanks for the reply. I did check that widget but it looks like the way it works is different. Because for the value I'm trying to implement it's using a proto buff with a set of options. What I'm struggling with is how to map a custom hex value with a string to a way that is going to be used by the proto buff. A nudge in the right direction would be helpful.

[MayurSMahajan]

Again I am not really familiar with this part of codebase, nor with protobuffs. Sorry about that!

[nebiyuelias1]

Again I am not really familiar with this part of codebase, nor with protobuffs. Sorry about that!

Ok who should I ask?

[MayurSMahajan]

Discord would be the right place to start, also your problem seems to be more related to Protobuffs, why do you want to implement it with Protobuffs, what advantage does it have over just reusing the existing solution? Just curious.

[nebiyuelias1]

Discord would be the right place to start, also your problem seems to be more related to Protobuffs, why do you want to implement it with Protobuffs, what advantage does it have over just reusing the existing solution? Just curious.

Honestly it's the way things work at the moment and implementing the feature request requires the modification I described.

[gustamor]

@nebiyuelias1 How's it going?

The custom color feature for the cursor and selection in the workspace settings adds an icon to the color text box. Clicking this icon opens the color picker, which returns the corresponding color code and places it in the text box. This action does not alter any other elements; it simply updates the text box value with the selected color code and submits it in the same way as if the user had manually entered the code. I think this might work the same way.

[nebiyuelias1]

@nebiyuelias1 How's it going?

The custom color feature for the cursor and selection in the workspace settings adds an icon to the color text box. Clicking this icon opens the color picker, which returns the corresponding color code and places it in the text box. This action does not alter any other elements; it simply updates the text box value with the selected color code and submits it in the same way as if the user had manually entered the code. I think this might work the same way.

Hi the situation is different in this context. More info about the problem could be found here on discord: https://discord.com/channels/903549834160635914/905299371485188137/1260206983122915388.
I didn't understand how the proto buff modifications should work so I haven't made much progress on this one. Would love to get some help!