Ticket #4136 (closed Bug: fixed)

Opened 4 years ago

Last modified 4 years ago

IE: Wrong attribute order

Reported by: arczi Owned by: fredck
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

4136.patch (3.5 KB) - added by tobiasz.cudnik 4 years ago.
4136_2.patch (4.1 KB) - added by fredck 4 years ago.

Change History

comment:1 Changed 4 years ago by tobiasz.cudnik

  • Status changed from new to assigned
  • Owner set to tobiasz.cudnik

Changed 4 years ago by tobiasz.cudnik

comment:2 Changed 4 years ago by tobiasz.cudnik

  • Keywords Review? IE added

comment:3 Changed 4 years ago by fredck

  • 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 4 years ago by fredck

comment:4 Changed 4 years ago by fredck

  • Keywords Review? added; Review- removed

comment:5 Changed 4 years ago by fredck

  • Status changed from assigned to new
  • Owner changed from tobiasz.cudnik to fredck

comment:6 Changed 4 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 4 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 4 years ago by fredck

  • Keywords Review? added; Review- removed

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

comment:9 Changed 4 years ago by garry.yao

  • Keywords Review+ added; Review? removed

comment:10 Changed 4 years ago by fredck

  • Status changed from new to closed
  • Resolution set to fixed

Fixed with [4017].

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