Opened 7 years ago

Closed 7 years ago

#5766 closed Bug (fixed)

‘ignoreEmptyParagraph’ behaves wrong

Reported by: Garry Yao Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.4.2
Component: Core : Output Data Version: 3.0
Keywords: Confirmed Cc:

Description

Reproducing Procedures

  1. Open the 'replacebyclass' sample page;
  2. Create several new paragraphs at the end of document;
  3. Switch to 'source' mode and back multiple times.
    • Actual Result: Each time with one paragraph lost at the end of document.
    • Expected Result: All paragraphs remains.

Attachments (2)

5766.patch (988 bytes) - added by Garry Yao 7 years ago.
5766_2.patch (1.3 KB) - added by Garry Yao 7 years ago.

Download all attachments as: .zip

Change History (10)

Changed 7 years ago by Garry Yao

Attachment: 5766.patch added

comment:1 Changed 7 years ago by Garry Yao

Owner: set to Garry Yao
Status: newassigned
Version: 3.0

We should give priority, look at the affected version.

comment:2 Changed 7 years ago by Garry Yao

Milestone: CKEditor 3.3CKEditor 3.4

comment:3 Changed 7 years ago by Garry Yao

Keywords: Review? added

comment:4 Changed 7 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.4CKEditor 3.5

comment:5 Changed 7 years ago by Frederico Caldeira Knabben

#5818 has been marked as DUP.

comment:6 Changed 7 years ago by Garry Yao

New patch adds full page handling, the drawback now is the original 'strip last empty' behavior is lost, but maybe not a drawback at all, as the major principle of an text editor is not touching user's content unnoticed.

Changed 7 years ago by Garry Yao

Attachment: 5766_2.patch added

comment:7 Changed 7 years ago by Sa'ar Zac Elias

Status: reviewreview_passed

comment:8 Changed 7 years ago by Garry Yao

Resolution: fixed
Status: review_passedclosed

Fixed with [5960].

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