skip unnecessary full-document replaces

yjs / y-prosemirror

ProseMirror editor binding for Yjs

https://demos.yjs.dev/prosemirror/prosemirror.html

MIT License

328 stars 115 forks source link

skip unnecessary full-document replaces #127

Open ocavue opened 1 year ago

ocavue commented 1 year ago

This PR skips some unnecessary replaces by comparing the incoming content and current content in the ProseMirror editor view. This could be a potential solution for https://github.com/yjs/y-prosemirror/issues/126

maccman commented 1 year ago

cc @dmonad :)

maccman commented 1 year ago

Ping :)

martinboksa commented 1 year ago

ping :)

fuadsaud commented 3 months ago

Has anyone verified this to fix #126? I rebase this PR on master and tried pointing my app to it, but I still see the same behavior (it's not impossible that I'm doing something wrong when building the library locally, given that I depend on y-prosemirror indirectly through tiptap).