Opened 3 years ago

Last modified 2 years ago

#14848 confirmed Bug

[IE8] link plugin test fails

Reported by: Marek Lewandowski Owned by:
Priority: Normal Milestone:
Component: General Version: 4.5.11
Keywords: Cc:


With the addition of #7154, the new link test case started to fail on IE8.

The issue is that a element applied by wraps also non-editable element, which is not the case in any other browser.

Change History (6)

comment:1 Changed 3 years ago by Marek Lewandowski

Status: newconfirmed

comment:2 Changed 3 years ago by Tade0

I checked out the commit that introduced this test and while it passes in Chrome, in IE8 it contains an error that results in a wait()-without-resume() exception.

comment:3 Changed 3 years ago by Tade0

Summary of findings:

Looking at the failed test case:

Values should be the same.
Expected: <a href="http://newlink"><em>foo </em>bbb <span contenteditable="false"><a href="aaa">ccc</a></span></a> (string)
Actual:   <a href="http://newlink"><em>foo </em>bbb </a><span contenteditable="false"><a href="aaa">ccc</a></span> (string)

One can notice that the selection "retreated" from the area occupied by the non-editable element. This is because while normally the selection engulfs the whole content, in IE8 it does not and on the right side it's anchored right after the a element inside the non-editable <span>(that a element is the end container).

In this specific situation the algorithm that applies inline styles treats the non-editable <span> as a no-go zone and does not apply the style there.

In conclusion: the problem lies in the selection, which does not span the whole content like it should.

comment:4 Changed 2 years ago by Marek Lewandowski

Milestone: CKEditor 4.6.0CKEditor 4.6.1

comment:5 Changed 2 years ago by Marek Lewandowski

Milestone: CKEditor 4.6.1CKEditor 4.6.2

comment:6 Changed 2 years ago by Marek Lewandowski

Milestone: CKEditor 4.6.2
Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy