Empty spans not being removed

[Marek Lewandowski]

occures at current master, was not present in 4.3 release

Extra, empty span is left in certian situations

1. open any sample with CKEditor (i.e. samples/replacebyclass.html)
2. set source to following html

   <h3>Type the title here</h3>
   
   <p>Type the text here</p>
   

3. go back to wysiwyg mode
4. open search dialog
5. type the text in find textfield
6. click find
7. close dialog

--- note already you have extra span left ---

You may now move cursor elsewhere and go back to the text substring, and span will still be there.

Expected result:
Produced source code:

<h3>Type the title here</h3>

<p>Type the text here</p>

Current result:
Code has extra span:

<h3>Type the title here</h3>

<p>Type <span>the text</span> here</p>

additional info:

1. you can insert multiple ammount of spans, by clicking find multiple times, none of them will be cleaned
2. i've reproduced it witch chrome and firefox @ Win8

[Piotrek Koszuliński]

​git:47c4514ba2e7225de2ec86d4eeec61b51adff5de is the first bad commit.

[Piotr Jasiun]

Changes in ​t/11258 and corresponding test branch.

[Piotr Jasiun]

What I do not like in this fix is the fact that I put changes in removeFromInsideElement. This method is called only from applyStyle but its name does not suggest this. I've added some description to removeFromInsideElement but I don't feel good about this.

[Piotrek Koszuliński]

Yes, it's not plainly clear, but the comment made it understandable. I'm ok with such solution. Because it makes sense and also, because there's no other way except reorganising big part of that code :).

[Piotrek Koszuliński]

I moved tests to style/applyremove. Those tests in style/editor are meant to test more than style system itself - they check if selection is ok and some other aspects of integration (like editor configuration).

I also pushed one commit to dev.

[Piotr Jasiun]

• ​git:59c2758
• tests:6c25949