Ckeditor5: Removing empty paragraph throws error

Created on 23 Aug 2016  Â·  4Comments  Â·  Source: ckeditor/ckeditor5

Browsers checked: Chrome

Steps to reproduce:

  • Go to manual test http://localhost:1030/build/modules/amd/tests/engine/manual/editingcontroller,
  • put cursor in the empty paragraph between foo and bar,
  • press Delete (Fn+Backspace on most Macs),

Results:

  • in the Console you will see error:

Uncaught CKEditorError: move-operation-nodes-do-not-exist: The nodes which should be moved do not exist.

  • when you try to move cursor up and down couple of times with arrow keys, you'll get another warning in the Console:

selectionchange-infinite-loop: Selection change observer detected an infinite rendering loop. undefined

engine bug

All 4 comments

I think that this test doesn't support Fn+delete (so forward delete). Looking at its code, it was only supposed to support backspace (which on some Macs is named "delete"). I'll fix the description.

I used Fn+Delete, because the backspace key was doing nothing.

Yeah, you're right. I've just checked the key codes precisely.

But this test is not supposed to work at the end of blocks – it's not what it checks. But this must be clarified, so I'll fix the test decription or use some feature to replace those mock.

I removed the whole test. It didn't make sense here, duplicating a lot of code which features introduce. Instead, I'll add simple manual tests for features which don't have them.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

Reinmar picture Reinmar  Â·  3Comments

MCMicS picture MCMicS  Â·  3Comments

msamsel picture msamsel  Â·  3Comments

pomek picture pomek  Â·  3Comments

oleq picture oleq  Â·  3Comments