UndoManager doesn't persist meta about yjs HOT 4 OPEN

The meta property is intended to store information about the "before-state" that is required to restore exactly that state. One example would be to store the cursor position that should be recovered when undo is triggered. When you undo several undo-items at once (e.g. because they are merged), you usually only want to recover the first meta state.

Hence, we intentionally discard the meta properties of stack items that are merged. Only the first meta generated should survive.

There might be an argument why stack-items should be merged. However, this would be confusing to most, as the first stack-item created should always be the "authority". I suggest that you store all relevant information on all meta states.

If this seems unfeasible to you, can you please explain your use-case?

from yjs.

First of all, apologies for the late reply.

The meta property is intended to store information about the "before-state" that is required to restore exactly that state. One example would be to store the cursor position that should be recovered when undo is triggered. When you undo several undo-items at once (e.g. because they are merged), you usually only want to recover the first meta state.

The issue we're having lies somewhere with this.

Let's consider an undo action:
When the stackItem is popped from undo the beforeState sync doesn't apply in time for the creation of the redo stackItem meta.

Shouldn't the previous meta be accessible for cases like this? Because when moving something from a stack to another more often than not you could be creating the same meta information

from yjs.

Let's say:

• Before performing a change, the cursor is at position X.
• Before performing the second change, the cursor position is at position Y.
• These two changes are merged into a single undo item.

When I hit undo, I expect that the cursor position is at position X.

I don't see a need to preserve the state "cursor it at position Y", because that change is merged into the first undo-stack item. We usually only want to restore a single state.

This is what the undo-manager was designed for. I'm not saying that we can't extend it. But I need to understand the problem first.

Now let's say we want to we want to preserve the meta of the second change. Clearly, we don't want to restore position X AND Y. So how exactly should "metas" be merged? Please be more specific.

This is an extremely generic ask and I honestly don't have an incentive to fix your specific problem. I'm not sure a solution would be beneficial to all.

from yjs.

Merging aside, and like you mentioned above, meta is supposed to store a "before-state" (which in redo it can be considered an"after-state")

Here's an example of what is trying to be done. When the user redos an action it should restore the position to where it was before the undo. No matter how long the user takes between the actions.

Also due to the current order of events (StackItem gets added before pop), it is not possible to just create a new meta directly with the correct position.

from yjs.