[IE11][FF][CF] Styling could not be reapplied on a simple click

[kkrzton]

Steps to reproduce

Can be easily reproduced on /tests/plugins/copyformatting/manual/safaricolor (for both editors).

1. Put caret inside first paragraph e.g. Text witho^ut any format.
2. Click Copy Formatting button.
3. Click on the last word (Orange) inside last paragraph (styling will be removed).
4. Click on the still styled word inside same paragraph e.g. Text with Bac^kground Colour Orange.
5. Click Copy Formatting button.
6. Click on the last word (Orange) inside last paragraph.

Expected result

On the second click styling should be copied, so word Orange should be styled same as rest of the paragraph.

Actual result

On the second click, styling is not applied.

Other details (browser, OS, CKEditor version, installed plugins)

IE11, FF (Chrome works as expected).

[kkrzton]

Applying styles for the first time (step 3) produces html like (on Firefox):

<p>
    <span style="background-color: #ff8c00;">Text with Background Colour </span>
    Orange
    <span style="background-color: #ff8c00;"></span>
    <br type="_moz">
</p>

In other browsers there is only empty span on the end, here is additional <br>. Those two empty elements causes some error in _getSelectedWordOffset function. With one empty element on the end, it works fine.

[kkrzton]

Pushed unit tests for this particular case ​ec3cf00.

[kkrzton]

Found similar issue but with different cause, extracted to #16718.

[kkrzton]

Pushed fix to ​t/16618 branch.

The code assumed that every (every used in a particular method) ​element node has a text child which was false for e.g. br nodes. I added an additional check.

[Tomasz Jakut]

The fix is not working correctly. It works only for empty span + br at the boundary of p. If the sentence is not inside paragraph, the plugin apply formatting to the wrong word.

Also only test applyFormat ending with empty span and br - #16618 really fails after disabling the fix.

Additionally test applyFormat ending with br - #16618 passes even after disabling fix in FF.

[kkrzton]

I can see that the fix does not work for some specific cases mentioned above.

Pushed ​additional tests covering those cases. Did some tests refactoring - now all tests but 2 fails on Firefox. These 2 tests just covers before untested case with space between text node and empty elements.

[Marek Lewandowski]

Let's focus on bringing it into 4.6.2.