Color picker has now a saturation and intensity triangle

scribble-rs / ScribblersPad

Scribble.rs Pad is a third party Scribble.rs client that is being developed in the Unity Game Engine.

MIT License

2 stars 0 forks source link

Color picker has now a saturation and intensity triangle #21

Closed BigETI closed 3 years ago

BigETI commented 3 years ago

This pull request replaces the color picker saturation and intensity sliders with a saturation and intensity triangle.

Bios-Marcel commented 3 years ago

So, when I extend the colorpalette, should I make it part of the API and should we make a different layout.

BigETI commented 3 years ago

Yes, but I would like not to add more colors to the pre-defined color palette, since we already have a lot of colors for users to choose from. Maybe allow defining color sets with fixed quantities within the API.

Since this is somewhat out of the scope of this pull request, should I merge this for now?

Bios-Marcel commented 3 years ago

Yeah, it's fine to merge. Either way, I saw sketchful.io having a set of common skin colors, which is kinda nice.