Opened 11 years ago

Closed 11 years ago

#3046 closed Bug (fixed)

plugin:list adding extra <br>

Reported by: Garry Yao Owned by: Martin Kou
Priority: Normal Milestone: CKEditor 3.0
Component: General Version: SVN (FCKeditor) - Retired
Keywords: Confirmed Review+ Cc:

Description

Both list commands appending extra <br> at end of the target block elements.

Attachments (3)

test-list-cancel2.patch (3.4 KB) - added by Garry Yao 11 years ago.
Functional test case
3046.patch (1.6 KB) - added by Martin Kou 11 years ago.
3046_2.patch (1001 bytes) - added by Martin Kou 11 years ago.

Download all attachments as: .zip

Change History (9)

Changed 11 years ago by Garry Yao

Attachment: test-list-cancel2.patch added

Functional test case

comment:1 Changed 11 years ago by Martin Kou

Keywords: Confirmed added
Owner: set to Martin Kou
Status: newassigned

Changed 11 years ago by Martin Kou

Attachment: 3046.patch added

comment:2 Changed 11 years ago by Martin Kou

Keywords: Review? added

The patch must be applied in conjunction with #3151's patch.

comment:3 Changed 11 years ago by Frederico Caldeira Knabben

Keywords: Review- added; Review? removed

There is already a function for it: CKEDITOR.dom.element::appendBogus.

Changed 11 years ago by Martin Kou

Attachment: 3046_2.patch added

comment:4 Changed 11 years ago by Martin Kou

Keywords: Review? added; Review- removed

comment:5 Changed 11 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

comment:6 Changed 11 years ago by Martin Kou

Resolution: fixed
Status: assignedclosed

Fixed with [3250].

Click here for more info about our SVN system.

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