Opened 16 years ago
Closed 16 years ago
#3679 closed Bug (fixed)
[IE8] element.html test fails in various places
Reported by: | Tobiasz Cudnik | Owned by: | Tobiasz Cudnik |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.0 |
Component: | General | Version: | |
Keywords: | IE Confirmed Review+ | Cc: |
Description (last modified by )
Depending on chosen mode on IE8, element.html test fails in various places. This patch seems to fix it, but it uses too many workarounds and need to be rewritten. It should use less workarounds as possible, mostly on IE8 strict.
Attachments (1)
Change History (7)
comment:1 Changed 16 years ago by
Owner: | set to Tobiasz Cudnik |
---|---|
Status: | new → assigned |
comment:2 Changed 16 years ago by
Description: | modified (diff) |
---|
comment:3 Changed 16 years ago by
Changed 16 years ago by
Attachment: | 3679.patch added |
---|
comment:4 Changed 16 years ago by
Keywords: | Review? added |
---|
comment:5 Changed 16 years ago by
Keywords: | Review+ added; Review? removed |
---|
There's still a minor problem for 'test_setAttribute2, test_setAttributes, test_getAttribute_ignoreExpandoAttributes' to pass on IE8 strict because of style text formatting problem:
*Expected:
float: right;*Actual:
FLOAT:RIGHT
But as discussed with Tobias, we'll open a new ticket for this.
comment:6 Changed 16 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
The correct fix would fall into setAttribute/removeAttribute, and addClass/removeClass just use the former under the hood.