Opened 10 years ago

Closed 10 years ago

#4136 closed Bug (fixed)

IE: Wrong attribute order

Reported by: Artur Formella Owned by: Frederico Caldeira Knabben
Priority: Normal Milestone: CKEditor 3.0
Component: QA Version:
Keywords: IE Review+ Cc:

Description

The following tests fail only because the attribute order

plugins/htmldataprocessor/htmldataprocessor.html:

Test named "test_ticket_3591_2" failed with message: "Values should be the same.".
Expected:
<object classid="clsid"><param name="movie" value="movie.swf" /><embed src="movie.swf" type="application/x-shockwave-flash"></embed></object>Actual:
<object classid="clsid"><param value="movie.swf" name="movie" /><embed type="application/x-shockwave-flash" src="movie.swf"></embed></object>

Test named "test_ticket_3884" failed with message: "Values should be the same.".
Expected:
<p><a href="" name="">emptylink</a></p>
Actual:
<p><a name="" href="">emptylink</a></p>

Style order - tests/plugins/styles/styles.html:

 Test named "test_inline16" failed with message: "First range".
Expected:
<b lang="pt" style="font-size:11pt;color:red;">this<b style="font-weight:700;"> is some sample text</b></b>Actual:
<b lang="pt" style="color:red;font-size:11pt;">this<b style="font-weight:700;"> is some sample text</b></b>

Attachments (2)

4136.patch (3.5 KB) - added by Tobiasz Cudnik 10 years ago.
4136_2.patch (4.1 KB) - added by Frederico Caldeira Knabben 10 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 10 years ago by Tobiasz Cudnik

Owner: set to Tobiasz Cudnik
Status: newassigned

Changed 10 years ago by Tobiasz Cudnik

Attachment: 4136.patch added

comment:2 Changed 10 years ago by Tobiasz Cudnik

Keywords: Review? IE added

comment:3 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: Review- added; Review? removed

I'm researching different solutions that has less impact in the TCs.

Btw... the styles issue is IE8 only.

Changed 10 years ago by Frederico Caldeira Knabben

Attachment: 4136_2.patch added

comment:4 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: Review? added; Review- removed

comment:5 Changed 10 years ago by Frederico Caldeira Knabben

Owner: changed from Tobiasz Cudnik to Frederico Caldeira Knabben
Status: assignednew

comment:6 Changed 10 years ago by Garry Yao

Keywords: Review+ added; Review? removed

There's yet another failed TC of styles test suite which is out of the scope of this ticket, is reported at #4147.

comment:7 Changed 10 years ago by Garry Yao

Keywords: Review- added; Review+ removed

It's noticed that CKEDITOR.test.getAttribute should receive the same treatment.

comment:8 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: Review? added; Review- removed

There is no current usage for changes on CKEDITOR.test.getAttribute.

comment:9 Changed 10 years ago by Garry Yao

Keywords: Review+ added; Review? removed

comment:10 Changed 10 years ago by Frederico Caldeira Knabben

Resolution: fixed
Status: newclosed

Fixed with [4017].

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