flutter_gradient_generator icon indicating copy to clipboard operation
flutter_gradient_generator copied to clipboard

Published 20 hours ago • verified publisher icon victoreronmosele

Store the color pallete states

Open maheshj01 opened this issue 1 year ago • 10 comments
trafficstars

Storing the states would allow the undo-redo feature to be able to switch between states and also compare between color palettes. To give you more context. I created a color palette and then hit Random and now I have no way to switch back.

Proposed solution:

  1. The app should show the list of states (ListView), clicking each listItem should switch to that state.
  2. This should also be binded with keyboard shortcuts for undo (ctrl+z) and redo (ctrl+shift+z)
  3. User should have option to clear the entire state (list) or a specific element in the list.

maheshj01 avatar Jan 07 '24 10:01 maheshj01

Thanks @maheshmnj. I agree and would like to have this feature in the project.

Can you describe some more how you imagine the flow to be?

victoreronmosele avatar Jan 07 '24 13:01 victoreronmosele

@victoreronmosele I have updated the description. I will see if I can contribute to this feature later this week.

maheshj01 avatar Jan 08 '24 04:01 maheshj01

I implemented this log history today to one of my app https://pastelog.netlify.app/ hopefully it gives you an idea

maheshj01 avatar Jan 08 '24 15:01 maheshj01

Thanks!

Your implementation of the state history on https://pastelog.netlify.app/ looks great!

I also like the idea for the keybindings.

I'm currently working on https://github.com/victoreronmosele/flutter_gradient_generator/issues/73 locally and I'll look deeper into this in a few days.

victoreronmosele avatar Jan 08 '24 16:01 victoreronmosele

@maheshmnj, I just merged in a UI update via https://github.com/victoreronmosele/flutter_gradient_generator/pull/75, and I added a toolbar that will contain undo and redo state buttons.

I'll do some work on this and share a preview soon.

I'll also appreciate some feedback on the UI update. See demo here: https://github.com/victoreronmosele/flutter_gradient_generator/pull/75#issue-2128576034.

victoreronmosele avatar Feb 12 '24 02:02 victoreronmosele

@victoreronmosele The UI looks great!

I have a few additonal improvements to suggest

  1. Clicking on black app bar reloads the app
  2. There seems some issue with the text rendering
image

maheshj01 avatar Feb 12 '24 21:02 maheshj01

@maheshmnj, thanks for the review and suggestions. 🙌🏽

I'll work on fixes for the issues.

victoreronmosele avatar Feb 12 '24 21:02 victoreronmosele

Added the fix for the empty space reloading the app.

The text rendering bug isn't so clear to me.

victoreronmosele avatar Feb 18 '24 23:02 victoreronmosele

@maheshmnj here's the current progress (undo and redo):

https://github.com/victoreronmosele/flutter_gradient_generator/assets/19398044/44f6ae38-3c3c-45bf-ae30-e7201eae9ebb

Preview URL: https://flutter-gradient-generator--pr79-state-history-pnd1e08v.web.app/ PR: https://github.com/victoreronmosele/flutter_gradient_generator/pull/79

Adding the version history next.

victoreronmosele avatar Feb 19 '24 03:02 victoreronmosele

Looks great!

maheshj01 avatar Feb 19 '24 04:02 maheshj01