Incorrect cursor position after undo

[philippkuehn]

Describe the bug
As soon as I create multiple nodes one after the other and want to undo this, the cursor stays in the last line. This issue can be reproduced on the official demo page.

To Reproduce
Steps to reproduce the behavior:

1. create paragraph
2. press enter
3. press cmd+z/ctrl+z

Expected behavior
Cursor should be in the first line instead of the second.

Screenshots
https://user-images.githubusercontent.com/2500670/108116413-ce668880-709b-11eb-89ea-dff5a41d6c47.mp4

[philippkuehn]

@dmonad ping 👀

[bedryck]

Hey, I have the same bug. Any updates?

[hamflx]

The following is my solution, mainly to change three places:

1. placing ystate within the undoManager event.
2. place the memory cursor position within _prosemirrorChanged (too many plug-ins are executed so that oldState does not really represent the last cursor position).
3. change the stack-item-popped event to stack-item-pop (I added this event myself, so you'll need to modify Yjs as well).

hamflx@e44952e

hamflx/yjs@716aa57

[Jordan-Ellis]

Any update on if there will be an official fix?

[romansp]

@hamflx thanks for the provided details and patches. After combining them all together with the patch from #43 (comment) I got it working flawlessly. It also restores different types of selections correctly.

By what it looks like all these patches together should resolve the following issues: #43, #101, #139.

@dmonad is it worth starting new PR to address them as it looks like #118 is stale?