[FF IE11] Shift+Enter in lists produces two line breaks

[Karen Ananiev]

Use the code:

<ul>
	<li><strong>test</strong></li>
	<li>test</li>
</ul>

Press Shift+Enter at the end of the first line.

Expected: One line created
Actual: Two lines created

[Piotrek Koszuliński]

I can't reproduce it.

• How do you test it step by step?
• On which version of CKEditor?
• On which version of Firefox?
• On which sample in which package (you can attach your build-config.js)?

[Karen Ananiev]

• Here is the screencast: ​http://www.screencast.com/t/YtE8fwlGD2l
• On ckeditor.com/demo as well as on nightly build
• FF 29.0

Please let me know if you need more details. Thank you!

[Piotrek Koszuliński]

Ok, I can see it. It must be the very first click. If caret is placed at the end by end button or if I click in more than one place before, I cannot reproduce it.

[Jakub Ś]

Problem can be reproduced from CKEditor 4.0 (works fine in beta). in IE11 and Firefox

[Jakub Ś]

I see that brs are inserted into strong tag and below the strong tag. I believe we had such issue with br tags being inserted into strong tag but I can't find it right now.

[Piotrek Koszuliński]

Hmm... that would mean that our element.getBogus method misses the <br> inserted inside <strong> by the browser. Or something similar. I have a feeling that this might be related to few other issues that we saw recently which happened at the end of line. Let's take a look at this - 4.4.2 it is.

[Jakub Ś]

I have found issue I was talking about #8958. This isn't the same case but as I remembered it concerns brs in strong :).

[Piotrek Koszuliński]

Pushed ​branch:t/11947. The bug was caused by enter key trying to append bogus to the wrong node. It did not consider a situation when line break is inserted into an inline element.

[Piotr Jasiun]

4 red tests in IE 11, on the t/11947 branch: ​http://tests.ckeditor.dev:1030/tests/plugins/enter/enterkey

[Piotrek Koszuliński]

Update benderjs-ckeditor.

[Piotrek Koszuliński]

Oops! I forgot to mention that you must use benderjs-ckeditor/t/11947.

[Piotr Jasiun]

Now everything looks and works fine. I rebased branch on the top on the master.

[Piotrek Koszuliński]

Fixed on master with ​git:c6b36c2 in ckeditor-dev and 62c81e0 in benderjs-ckeditor.