search

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

Cut is buggy #97

Open TiPlayer opened 8 years ago

TiPlayer commented 8 years ago

This are a lot of bugs related to cut. Like if we add a character at the end of the text, and we select all and press ctrl-x, all the text are gone. And if we and multiple text at different place, and we hit ctrl-x, ctrl-z quickly, we will get something like: 2016-03-01 9 42 15 2016-03-01 9 42 24

And also I notice other really severe bugs related to cut. But those do not exist when we use copy & delete. So can you change Cut to copy & delete?

imdfl commented 8 years ago

Thanks for the detailed report. I doubt that cut can be changed to copy+delete, because the core function is performed by the browser with no involvement by CKEditor. As far as I know, you can't re-route the browser action of 'cut' to 'copy', at least not in a cross-browser way.

TiPlayer commented 8 years ago

So is there a better way to solve this? Another bug is that, if there is a tag at the end of the selection, and if you do a cut action, all the selection will be surrounded by the tag after cut. 2016-03-01 10 00 14 2016-03-01 10 00 24

imdfl commented 8 years ago

I (or whoever wants to be involved) need to debug this flow and identify the parts of the problem that stem from buggy LITE code.