loopindex / ckeditor-track-changes

Track changes plugin for CKEditor, based on ICE (track changes for TinyMCE).
www.loopindex.com
Other
51 stars 55 forks source link

Empty tables disappears #130

Open ppecio opened 6 years ago

ppecio commented 6 years ago

When i add new table in content which is not makred as a change (is a black regular text), click somewhere out of the editor and then click back into editor (but not on change) table disappears. After some investigation I found that selection cleaning up function is causing this, which calls _cleanupAroundNode where dom.hasNoTextOrStubContent returns true for insertion/deletion with empty table inside.

Can be easily reproduced on your demo page on all major browsers.

imdfl commented 6 years ago

Yes, the cleanup is too aggressive. Not sure I have the resource to address this now, but I'll keep it in mind (or wait for the community to step in...)