Skip to content
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

Editor Undo/Redo #694

Closed
2 tasks done
baconpaul opened this issue Mar 2, 2019 · 5 comments · Fixed by #6079
Closed
2 tasks done

Editor Undo/Redo #694

baconpaul opened this issue Mar 2, 2019 · 5 comments · Fixed by #6079
Labels
Before Beta If these are open we can't really call ourselves beta Design Required We need to design a solution to this issue Feature Request New feature request UX Issues related to user experience (UX) - mouse, touch, keyboard, MIDI inputs, etc.
Milestone

Comments

@baconpaul
Copy link
Collaborator

baconpaul commented Mar 2, 2019

OK tackle this so here's the checklist. As always items on the list without a check are known and not done; items with a check are in a branch which isn't merged or pr yet

  • Undo accessibility and announcements
  • Undoable action clears redo stack

Would be great if the synth had undo/redo for edits. from
https://www.kvraudio.com/forum/viewtopic.php?p=7330177#p7330177

@baconpaul baconpaul added the Feature Request New feature request label Mar 2, 2019
@baconpaul baconpaul added this to the 1.6.n milestone Mar 2, 2019
@baconpaul baconpaul changed the title Undo/Redo Editor Undo/Redo Mar 2, 2019
@baconpaul baconpaul modified the milestones: 1.6.n, Currently Unscheduled Oct 4, 2019
@mkruselj mkruselj added the UX Issues related to user experience (UX) - mouse, touch, keyboard, MIDI inputs, etc. label Feb 5, 2020
@j5v
Copy link
Contributor

j5v commented Jul 29, 2020

Came to post, but found this issue was already here.

  • Undo and Redo could go next to Store in the UI.
  • Storage: happy with session persistence only.
  • When to store:
    • on data entry,
    • on mouse up on slider, or
    • 2s delay after scroll action, or overwrite the last history state if changing the same parameter again.
  • Keyboard shortcuts: use Ctrl/Cmd+Z and Ctrl/Cmd+Y (or Shift+Ctrl/Cmd+Z), preventing pass-thru to host.

@mkruselj
Copy link
Collaborator

  • Keyboard shortcuts: use Ctrl/Cmd+Z and Ctrl/Cmd+Y (or Shift+Ctrl/Cmd+Z), preventing pass-thru to host.

Unfortunately this one would not be happening, as dealing with keyboard events in all the various DAWs is just a humongous pain in the ass (and sometimes just doesn't work no matter what anyways)

@mkruselj mkruselj added the Design Required We need to design a solution to this issue label Sep 21, 2021
@mkruselj
Copy link
Collaborator

Apparently, moving to XT 1.1 milestone. 🥳

baconpaul added a commit to baconpaul/surge that referenced this issue Apr 1, 2022
This is a basic infrastucture which moves us towards undo
in an editor. Lots missing here all detailed in a checklist
in surge-synthesizer#694 but with this commit parameter changes, modulation changes,
osc and fx changes all push into the undo stack
baconpaul added a commit to baconpaul/surge that referenced this issue Apr 1, 2022
This is a basic infrastucture which moves us towards undo
in an editor. Lots missing here all detailed in a checklist
in surge-synthesizer#694 but with this commit parameter changes, modulation changes,
osc and fx changes all push into the undo stack. Use a simple limit
of 250 gestures per instance captured.
baconpaul added a commit that referenced this issue Apr 1, 2022
This is a basic infrastucture which moves us towards undo
in an editor. Lots missing here all detailed in a checklist
in #694 but with this commit parameter changes, modulation changes,
osc and fx changes all push into the undo stack. Use a simple limit
of 250 gestures per instance captured.
@mkruselj
Copy link
Collaborator

mkruselj commented Apr 2, 2022

  • "Undo changes" action shows both on top and on bottom of Edit Keybindings overlay.
  • Redo action?

@baconpaul
Copy link
Collaborator Author

Ahh I forgot about redo. Will add this week before I add more undoables

baconpaul added a commit to baconpaul/surge that referenced this issue Apr 6, 2022
Add redo support

- Params can redo
- Redo for Modulations
- FX and OSC work
- Stack dump useful and in dev menu

Addresses surge-synthesizer#694
@baconpaul baconpaul mentioned this issue Apr 6, 2022
baconpaul added a commit to baconpaul/surge that referenced this issue Apr 6, 2022
Add redo support

- Params can redo
- Redo for Modulations
- FX and OSC work
- Stack dump useful and in dev menu
- Debug build label moves and dev menu on

Addresses surge-synthesizer#694
baconpaul added a commit that referenced this issue Apr 6, 2022
Add redo support

- Params can redo
- Redo for Modulations
- FX and OSC work
- Stack dump useful and in dev menu
- Debug build label moves and dev menu on

Addresses #694
baconpaul added a commit to baconpaul/surge that referenced this issue Apr 7, 2022
Addresses surge-synthesizer#694

- Step Sequencer undo
baconpaul added a commit to baconpaul/surge that referenced this issue Apr 7, 2022
Step Sequencer Udno. Also lays out the pattern for the other
compound types.

Addresses surge-synthesizer#694
baconpaul added a commit that referenced this issue Apr 7, 2022
Step Sequencer Udno. Also lays out the pattern for the other
compound types.

Addresses #694
baconpaul added a commit to baconpaul/surge that referenced this issue Apr 7, 2022
- Swap LFO display on LFO undo
- Rename LFO and Macro
- Macro Values
- LFO Type Undo/Redo
- Modulation List Gestures in Undo/Redo
- An Undo/Redo button

Addresses surge-synthesizer#694
mkruselj added a commit to mkruselj/surge that referenced this issue Apr 9, 2022
Right click on undo/redo shows context menu with link to manual and entry to open Action History overlay
Implement scaffolding for Action History overlay
Remove unnecessary break statements in createOverlay() method
Hovers are not displayed for Switches if they're deactivated
Add new assets to SkinImageMaps res[] array

Last but not the least, do a tiny bit of renaming for AW Cabs

Addresses surge-synthesizer#694
mkruselj added a commit that referenced this issue Apr 9, 2022
* Finalize undo history assets and more

Right click on undo/redo shows context menu with link to manual and entry to open Action History overlay
Implement scaffolding for Action History overlay
Remove unnecessary break statements in createOverlay() method
Hovers are not displayed for Switches if they're deactivated
Add new assets to SkinImageMaps res[] array

Last but not the least, do a tiny bit of renaming for AW Cabs

Addresses #694
baconpaul added a commit to baconpaul/surge that referenced this issue Apr 10, 2022
Addresses surge-synthesizer#694

- Modulator mute undoes
- Modulator menu items work
- MSEG
baconpaul added a commit that referenced this issue Apr 10, 2022
Addresses #694

- Modulator mute undoes
- Modulator menu items work
- MSEG
baconpaul added a commit to baconpaul/surge that referenced this issue Apr 11, 2022
baconpaul added a commit that referenced this issue Apr 11, 2022
@mkruselj mkruselj added the Before Beta If these are open we can't really call ourselves beta label Apr 18, 2022
baconpaul added a commit to baconpaul/surge that referenced this issue Apr 22, 2022
UI gestores which change tuning (drag and drop and editor)
participate in undo/redo

Addresses surge-synthesizer#694
baconpaul added a commit to baconpaul/surge that referenced this issue Apr 22, 2022
UI gestores which change tuning (drag and drop and editor)
participate in undo/redo. Applying a Formula puts it in the
undo stack (but the code editor component eats Ctrl-Z for micro
changes so it is subtle).

Addresses surge-synthesizer#694

f
baconpaul added a commit that referenced this issue Apr 22, 2022
UI gestores which change tuning (drag and drop and editor)
participate in undo/redo. Applying a Formula puts it in the
undo stack (but the code editor component eats Ctrl-Z for micro
changes so it is subtle).

Addresses #694
baconpaul added a commit to baconpaul/surge that referenced this issue Apr 22, 2022
- Redo/Undo through patch changes

Addresses surge-synthesizer#694
baconpaul added a commit to baconpaul/surge that referenced this issue Apr 23, 2022
- Redo/Undo through patch changes
- Have an approximate undo/redo memory threshold
- Trip undo/redo based on being over thresholds
- Cleanup memory here and there from the patch thing

Addresses surge-synthesizer#694
baconpaul added a commit that referenced this issue Apr 23, 2022
- Redo/Undo through patch changes
- Have an approximate undo/redo memory threshold
- Trip undo/redo based on being over thresholds
- Cleanup memory here and there from the patch thing

Addresses #694
baconpaul added a commit to baconpaul/surge that referenced this issue Apr 23, 2022
1. Parameter temposync etc
2. Full LFO on LFO menu reset

Addresses surge-synthesizer#694
baconpaul added a commit to baconpaul/surge that referenced this issue Apr 23, 2022
1. Parameter temposync etc
2. Full LFO on LFO menu reset

Addresses surge-synthesizer#694
baconpaul added a commit to baconpaul/surge that referenced this issue Apr 23, 2022
1. Parameter temposync etc
2. Full LFO on LFO menu reset

Addresses surge-synthesizer#694
baconpaul added a commit that referenced this issue Apr 23, 2022
1. Parameter temposync etc
2. Full LFO on LFO menu reset

Addresses #694
baconpaul added a commit to baconpaul/surge that referenced this issue Apr 23, 2022
So do do undo undo redo -> stack OK
do do undo newdo -> redostack clear

Addreses surge-synthesizer#694
baconpaul added a commit that referenced this issue Apr 23, 2022
So do do undo undo redo -> stack OK
do do undo newdo -> redostack clear

Addreses #694
baconpaul added a commit to baconpaul/surge that referenced this issue Apr 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Before Beta If these are open we can't really call ourselves beta Design Required We need to design a solution to this issue Feature Request New feature request UX Issues related to user experience (UX) - mouse, touch, keyboard, MIDI inputs, etc.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants