conclave-team / conclave

CRDT and WebRTC based real-time, peer-to-peer, collaborative text editor
http://conclave.tech
MIT License
710 stars 130 forks source link

Fixed line wrapping issue #31

Closed unsudo closed 3 years ago

unsudo commented 4 years ago

Applying remote operations that require to insert or delete a line is not working properly on some browsers. Remote cursors are also not functioning properly as a result.

Adding a zero width-space text node after the cursor node fixes these issues.

sunny-b commented 4 years ago

Oh thanks for the PR! Sorry for the late response, I don't check this repo too often. Let me check out your change and test it and I'll get back to you about merging it in.

almereyda commented 3 years ago

I've had the same issue when testing conclave, and am happy to see that there is a solution to it.

sunny-b commented 3 years ago

Tested this out locally and looks good! Thanks for the PR