Undo in a property block permanently erases all edits
Nobody has claimed this yet.
- Dominant language
- Clojure
- Stars
- 45k
- Forks
- 2.8k
- Avg merge
- 1d 22h
- Merged PRs (30d)
- 105
Description
Search first
- I searched and no similar issues were found
What Happened?
When editing multiple properties in a property block, if you hit the UNDO shortcut key all edits are erased. (I think the expected behavior should be to just erase the edits on the current line/property).
This video demonstrates this behavior on the mac in logseq 0.8.18: https://drive.google.com/file/d/11nkvUQsI6JD1lEUeuaWhu_-CWsm70Ppi/view?usp=share_link
I neglected to show this in the recording, but when this happens REDO does not work. The edits are permanently lost.
Reproduce the Bug
- Open a template that has a block with multiple properties on it.
- Click into the properties area and start adding values for the several properties
- Without leaving edit mode, hit the keyboard shortcut for UNDO
- All edits will be erased
- Hit the keyboard shortcut for REDO.
- No edits are restored
Expected Behavior
I would hope that just the edits in the current property would get erased.
I would hope that hitting REDO would restore the edits just made in the current property.
Screenshots
https://drive.google.com/file/d/11nkvUQsI6JD1lEUeuaWhu_-CWsm70Ppi/view?usp=share_link
Desktop or Mobile Platform Information
macOS 12.6.3, logseq 0.8.18
Additional Context
No response
Are you willing to submit a PR? If you know how to fix the bug.
- I'm willing to submit a PR (Thank you!)
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
First reproduce the issue in the property-block editor on macOS using the listed six-step sequence, then trace the editor's undo and redo entry points. Done means undo removes only the current property's edits and redo restores them without erasing other property edits.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- clojure
- Domain
- desktop, frontend
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100