Opened 8 years ago
Last modified 8 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: |
Description
With the addition of #7154, the new link test case started to fail on IE8.
The issue is that a
element applied by CKEDITOR.style
wraps also non-editable element, which is not the case in any other browser.
Change History (6)
comment:1 Changed 8 years ago by
Status: | new → confirmed |
---|
comment:2 Changed 8 years ago by
comment:3 Changed 8 years ago by
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 8 years ago by
Milestone: | CKEditor 4.6.0 → CKEditor 4.6.1 |
---|
comment:5 Changed 8 years ago by
Milestone: | CKEditor 4.6.1 → CKEditor 4.6.2 |
---|
comment:6 Changed 8 years ago by
Milestone: | CKEditor 4.6.2 |
---|
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.