[IE11] Cursor jump to the top of the editor

[mencia]

Steps to reproduce

1. Change the editor to the source code mode.
2. Copy and paste the contents in the attached file.
3. Change the editor to the WYSIWYG mode.
4. Scroll down the editor viewport to the bottom.
5. Left mouse click on the last 'brownie.jpg' image.
6. Cursor jump to the top of the editor viewport.

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

[CKEditor version]

• ckeditor_4.6.2_full
• ckeditor_full_nightly
• CKEditor Nightly Build | Demo Full ​http://nightly.ckeditor.com/17-02-07-07-08/full/samples/

[Client OS, Browser]

• Windows 10 Pro(64bit) / Internet Explorer 11

[mencia]

Expected result

The image should be selected.

Actual result

Cursor jump to the top of the editor viewport.

[Jakub Ś]

Problem can be reproduced from CKEditor 4.6.0 in IE9-11.

[Marek Lewandowski]

We need to bisect it first to understand when the bug was introduced.

[Tade0]

Observation: clicking on the image scrolls the editor, while clicking next to it but close enough for the cursor to change into the move icon simply selects the image.

[Tade0]

Observation: This does not happen in all editor configurations.

There's something special in the way that last image is made.

[Tade0]

As with any bug of this type opening the dev tools is enough to make it disappear.

[Tade0]

Bisecting revealed that this is the offending commit: ​https://github.com/ckeditor/ckeditor-dev/commit/9f97b190fcaab80780e720dd1c7681c60d9f7dea

So apparently there's a problem somewhere in the copyformatting plugin.

[Tade0]

This particular problem can be solved by setting the focusEditor flag in the applyFormatting command in CF to false.

Unit tests pass - checking if it doesn't break anything else.

[Tade0]

Checked enough manual tests(especially ones concerning focus) to be confident that nothing has been broken by this change.

Added manual test.

Changes pushed to ​branch:t/16845.

[Tomasz Jakut]

LGTM. Merged with ​git:a4e9eef.