Skip to content

3D: Fix editor undo/redo keybinds and hotkey signal bugs#1252

Open
tanish2k09 wants to merge 1 commit intomainfrom
bugfix/3d-undo-redo
Open

3D: Fix editor undo/redo keybinds and hotkey signal bugs#1252
tanish2k09 wants to merge 1 commit intomainfrom
bugfix/3d-undo-redo

Conversation

@tanish2k09
Copy link
Copy Markdown
Collaborator

  • Ctrl+Z/Cmd+Z was calling redo() instead of undo(); fixed and added missing Ctrl+Shift+Z/Cmd+Shift+Z and Ctrl+Y redo bindings
  • Wrap all async keybind handlers in try/finally so isProcessing is always cleared even if undo/redo/copy/paste throws
  • Fix === typo in disableHotkeyInput (comparison instead of assignment), and use full value replacement so the Preact signal update is reactive

- Ctrl+Z/Cmd+Z was calling redo() instead of undo(); fixed and added
  missing Ctrl+Shift+Z/Cmd+Shift+Z and Ctrl+Y redo bindings
- Wrap all async keybind handlers in try/finally so isProcessing is
  always cleared even if undo/redo/copy/paste throws
- Fix === typo in disableHotkeyInput (comparison instead of assignment),
  and use full value replacement so the Preact signal update is reactive
@netlify
Copy link
Copy Markdown

netlify bot commented Mar 30, 2026

Deploy Preview for genhub-website ready!

Name Link
🔨 Latest commit a71b0a8
🔍 Latest deploy log https://app.netlify.com/projects/genhub-website/deploys/69ca94b47150d70008b0c044
😎 Deploy Preview https://deploy-preview-1252--genhub-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 30, 2026

Deploy Preview for artcraft-website ready!

Name Link
🔨 Latest commit a71b0a8
🔍 Latest deploy log https://app.netlify.com/projects/artcraft-website/deploys/69ca94b4dc3c010008571c8e
😎 Deploy Preview https://deploy-preview-1252--artcraft-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 30, 2026

Deploy Preview for storyteller-docs ready!

Name Link
🔨 Latest commit a71b0a8
🔍 Latest deploy log https://app.netlify.com/projects/storyteller-docs/deploys/69ca94b40ce1720008290fe2
😎 Deploy Preview https://deploy-preview-1252--storyteller-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@tanish2k09 tanish2k09 linked an issue Mar 30, 2026 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3D: Undo/Redo broken at least on MacOS

1 participant