Delete pressed in table cell causes an error when magicline is visible

[Piotrek Koszuliński]

Use this HTML:

<table border="1" cellpadding="1" cellspacing="1" style="width:500px">
	<tbody>
		<tr>
			<td>
			<p>&nbsp;</p>
			</td>
			<td>&nbsp;</td>
		</tr>
	</tbody>
</table>

1. Place caret in first table cell.
2. Move your mouse cursor to a place which will trigger magicline.
3. Press delete.
4. Error is thrown.

Uncaught TypeError: Cannot read property 'type' of null range.js:1526
CKEDITOR.dom.range.setStart range.js:1526
CKEDITOR.dom.range.setStartAfter range.js:1573
CKEDITOR.dom.range.setStartAt range.js:1618
CKEDITOR.dom.range.moveToPosition range.js:1489
CKEDITOR.dom.range.moveToElementEditablePosition range.js:2047
CKEDITOR.dom.range.moveToElementEditEnd range.js:2157
(anonymous function)

Starts in: ​https://github.com/ckeditor/ckeditor-dev/blob/c240965c7134620f8d2fe2c36f865197dc280aa9/core/editable.js#L653

Reproduced it on every browser.

[Jakub Ś]

I was trying to reproduce this for fair amount of time and nothing happened in CKEditor 4.4. Could you:

1. Tell me if you are still able to reproduce it?
2. Should this table be only HTML in editor?
3. How do you pace caret in table, just with click?
4. Is there anything you might have missed?

[Piotrek Koszuliński]

Without any additional effort I reproduced in on Chrome. Just copy this HTML to source, switch to wysiwyg, place caret in first cell, press delete and boom.

[Piotrek Koszuliński]

Hmm... I can't reproduce this on Chrome on Mac. We need to check it on more machines.

[Marek Lewandowski]

I confirm it in FF, Chrome, Chromium@Win7.

FF thorws:

TypeError: startNode is null

[Olek Nowodziński]

I'm not able to reproduce it. Tried Chrome on Mac and Win and IE9.

[Piotrek Koszuliński]

Since Marek reproduced it too, I'll give it a try to see what it may be. I may understand then why only some of us can reproduce it.

[Piotrek Koszuliński]

OK, I figured this out. You need to have magicline visible while pressing backspace :D.

[Olek Nowodziński]

Replying to Reinmar:

OK, I figured this out. You need to have magicline visible while pressing backspace :D.

The problem would be gone if we used lineutils as an engine of magicline (#9400, #10933). I hope, one day, it becomes a reality :)