Opened 8 years ago

Closed 8 years ago

#4144 closed Task (fixed)

htmlParser TC fix

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

Description

htmlParser test case fix in attachment.

Attachments (2)

4144.patch (769 bytes) - added by Artur Formella 8 years ago.
4144_2.patch (1.1 KB) - added by Garry Yao 8 years ago.

Download all attachments as: .zip

Change History (7)

Changed 8 years ago by Artur Formella

Attachment: 4144.patch added

comment:1 Changed 8 years ago by Artur Formella

Keywords: Review? added
Owner: set to Artur Formella
Status: newassigned

comment:2 Changed 8 years ago by Garry Yao

Component: QACore : Styles
Keywords: Review- added; Review? removed
Owner: changed from Artur Formella to Garry Yao
Status: assignednew

The changes are all valid, but it's still not passing test_fromHtml_3715 yet.
I've found a html parser error with pre, I'm taking over the ticket.

Changed 8 years ago by Garry Yao

Attachment: 4144_2.patch added

comment:3 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review- removed

I think you forgot requesting review.... here you have it.

comment:4 Changed 8 years ago by Garry Yao

Status: newassigned

comment:5 Changed 8 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed

Fixed with [4010] and [4011].

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