Skip to content

fix(hud): make undo and redo step through one action at a time - #1364

Open
Tellinq wants to merge 1 commit into
Polyfrost:v1from
Tellinq:fix/undo-redo
Open

Tellinq wants to merge 1 commit into
Polyfrost:v1from
Tellinq:fix/undo-redo

Conversation

@Tellinq

@Tellinq Tellinq commented Nov 13, 2026

Copy link
Copy Markdown
Contributor

Description

Undo in the HUD editor could jump back to the state from when the editor was opened instead of just the previous action.

  • Options holding arrays (like colors) never matched their own snapshot, so edits were never recorded. They are now compared by content.
  • Drags started right after another one were merged into a single step. Each gesture is now its own step.

Related Issue(s)

Fixes #1305

Checklist

  • I made a clear description of what was changed
  • I stated why these changes were necessary
  • I updated documentation or said what needs to be updated
  • I made sure these changes are backwards compatible
  • This pull request is for one feature/bug fix

Options holding arrays, like colors, never compared equal to their own
snapshot, so edits were never recorded and undo jumped back to the editor
opening. Quick back-to-back drags were also merged into a single step.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

bug: undoing in hud editor

1 participant