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.