[Webkit] Enter key before sub list is not right

[Piotrek Koszuliński]

<ol>
	<li>item1</li>
	<li>&nbsp;
	<ol>
		<li>item2</li>
	</ol>
	</li>
</ol>

Place caret at the beginning of second list item (nbsp) and press enter.

Result:

<ol>
	<li>^item1</li>
	<li>&nbsp;</li>
	<li>&nbsp;
	<ol>
		<li>item2</li>
	</ol>
	</li>
</ol>

Note that the caret was moved up to the item 1.

I was able to reproduce this on Chrome, but not on FF.

[Piotrek Koszuliński]

Expected result:

<ol>
	<li>item1</li>
	<li>&nbsp;</li>
	<li>^&nbsp;
	<ol>
		<li>item2</li>
	</ol>
	</li>
</ol>

[Jakub Ś]

Problem can be reproduced on both Chrome and Safari in Win and Mac from CKEditor 4.0 (4.0 beta works fine).

[Piotrek Koszuliński]

DUP reported: #10810.

[Piotrek Koszuliński]

Another DUP: #6448.

[Satya Minnekanti]

cc

[Olek Nowodziński]

​git:17798bb0e282a is the first bad commit.

[Piotrek Koszuliński]

I suppose it may be a filling char which should be removed before the action or skipped by some algorithm. Good luck :)

[Olek Nowodziński]

Pushed two tests to dt/plugins/enter/list.html (t/10247).

Recent discoveries:

• Removing the ​following line makes all tests green.

It feels like the correct way to fix the issue provided a right condition to bypass that removal is found.

I'm not sure whether removing it totally would be a good idea because we may experience unknown regressions (despite of passing tests). On the other hand, since that line belongs to #1272, which I'm unable to re-produce, it may be that ​this block is nothing but a little troublemaker and a dead piece of code. We need to check this out.

• Removing ​this condition (introduced by the first bad commit) also makes dt/plugins/enter/list.html green but some of dt/core/selection/editor.html fail. IMO it's nothing like a good solution.