Undo of custom delete button scrolls to from position rather than keeping affected node in view

1

Is there some way we can either disable or override the scrollIntoView found on this line? https://github.com/ProseMirror/prosemirror-history/blob/40d274a74d0fc0787aeca03634a64d0c78f18a50/src/history.ts#L341

I think an ideal solution would be to only scrollIntoView if the selection actually changes as a result of the history transaction. I believe that would remedy our problem. What do you think @marijn?

2

@marijn sorry to ping you. I wonder if you have any thoughts on this?

My suggestion is to patch the scrollIntoView logic to only scroll the view when the from position has actually changed. The current logic assumes that changes to the document will always occur at the from position but that’s not necessarily true.