Ckeditor5: Page will be crashed if I'll do right click

[dmitrykinakh]

See the video: http://take.ms/maq61

Also, I'm attaching source code of that table which might help you guys to determine an issue.
table_crash.txt

[Reinmar]

@jodator, it seems to be related to https://github.com/ckeditor/ckeditor5-table/issues/11

I think we need to report it to Chrome. Naturally, a browser crash is not a problem on our side.

[jodator]

@Reinmar yeah it looks like it's the case - anyway the https://github.com/ckeditor/ckeditor5-table/pull/39 fixes the ckeditor/ckeditor5#3167 AFAICS so this one might be fixed also.

[Reinmar]

That's great. Although, we simply prevent the native issue from happening by not letting the selection span editable boundaries.

PS. @dmitrykinakh, great to see the new table plugin being tested in an app before we released it :)

[jodator]

@Reinmar: I cannot reproduce it on t/12 branch while on master I can.

[dmitrykinakh]

@Reinmar, we have much more work to do with CKEditor 5. Table feature is only one small thing we have added. The next one probably would be collaborative editing.

PS: Hello from Lutsk, Ukraine ;-)

[dmitrykinakh]

@jodator, @Reinmar - I still can reproduce this in Chrome (not in Firefox) even I'm using t/12 branch.
But now I've strict steps how I can reproduce this, see the video - http://take.ms/HRG82

[Reinmar]

The clue of this change is in https://github.com/ckeditor/ckeditor5-engine/pull/1431. Which is not merged yet (unlike the PR in ckeditor5-table). You would need to check it with the changes in ckeditor5-engine.

[jodator]

@dmitrykinakh You should use also branch t/ckeditor5-table/12 in ckeditor5-engine as the actual fix is there. It's going to land on master in the matter of days.

[jodator]

@dmitrykinakh could you check your issue with current CKEditor? The selection changes landed in the CKEditor v10.1.0 release so it should work now as expected. Please reopen if you it does not work.

I'm closing this as a dup of ckeditor/ckeditor5#3167.