jb-tlm / color-charter

https://color-charter.vercel.app
MIT License
0 stars 0 forks source link

change history buttons #10

Closed jb-tlm closed 1 year ago

jb-tlm commented 1 year ago

"History" buttons should be changed to "Add to/Remove from Palette" buttons. My reasoning is that this tool doesn't actually store any of the colors (as soon as you refresh the page the "History" colors disappear), so that description may be misleading for users. Second, I would love to see a "copy color palette" button on that "History" bar of colors so that users can create their own color palettes using these stored values. image

marcioreisjr commented 1 year ago

Hello, I need clarification; Is the phrase 'Second, I would love to see a "copy color palette" button on that "History" bar of colors so that users can create their own color palettes using these stored values' meaning implementing what we did at #16? If not, then what does "these stored values" exactly mean (its data model)?

jb-tlm commented 1 year ago

Currently that copy button copies the color to the clipboard, and the history+ button adds this color to the palette at the top. Instead of the clipboard being in the bottom right and only clipboarding the currently selected color, they want it moved up to the end of the palette and to have it copy all the colors from the palette. Then, basically they just want the names of the 2 buttons changed. They think "history" is misleading like the app will save your palette when it doesn't... if you quit and come back it's gone, if you refresh it's gone. So they want these buttons to simply say "Add to Palette" and "Remove from Palette"