-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Store the color pallete states #74
Comments
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 I have updated the description. I will see if I can contribute to this feature later this week. |
I implemented this log history today to one of my app https://pastelog.netlify.app/ hopefully it gives you an idea |
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 #73 locally and I'll look deeper into this in a few days. |
@maheshmnj, I just merged in a UI update via #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: #75 (comment). |
@victoreronmosele The UI looks great! I have a few additonal improvements to suggest
|
@maheshmnj, thanks for the review and suggestions. 🙌🏽 I'll work on fixes for the issues. |
Added the fix for the empty space reloading the app. The text rendering bug isn't so clear to me. |
@maheshmnj here's the current progress (undo and redo): flutter_gradient_generator_undo_redo.mp4Preview URL: https://flutter-gradient-generator--pr79-state-history-pnd1e08v.web.app/ Adding the version history next. |
Looks great! |
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:
The text was updated successfully, but these errors were encountered: