Opened 8 years ago

Closed 8 years ago

#4139 closed Bug (fixed)

TCs in plugins/list are broken

Reported by: Garry Yao Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.0
Component: Core : Styles Version:
Keywords: Confirmed Review+ Cc:

Description

Regression of [3894] and [3903] break the test.

Attachments (2)

4139.patch (2.3 KB) - added by Garry Yao 8 years ago.
4139_2.patch (2.4 KB) - added by Garry Yao 8 years ago.

Download all attachments as: .zip

Change History (8)

Changed 8 years ago by Garry Yao

Attachment: 4139.patch added

comment:1 Changed 8 years ago by Garry Yao

Keywords: Review? added
Status: newassigned

comment:2 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: Review- added; Review? removed

The tests are passing on IE only with the patch. Don't pass in FF and Safari.

Changed 8 years ago by Garry Yao

Attachment: 4139_2.patch added

comment:3 Changed 8 years ago by Garry Yao

Keywords: Review? added; Review- removed

#3872 is not reproduced any more at this point, so [3903] which break the TC is reverted.

comment:4 in reply to:  3 Changed 8 years ago by Garry Yao

Replying to garry.yao:

#3872 is not reproduced any more at this point, so [3903] which break the TC is reverted.

Typo #3872 -> #3782.

comment:5 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

Ok... I'm leaving the Safari remaining failure to another ticket.

comment:6 Changed 8 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed

Fixed with [4022].

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