Ticket #5766 (closed Bug: fixed)

Opened 5 years ago

Last modified 4 years ago

‘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

5766.patch (988 bytes) - added by garry.yao 5 years ago.
5766_2.patch (1.3 KB) - added by garry.yao 4 years ago.

Change History

Changed 5 years ago by garry.yao

comment:1 Changed 5 years ago by garry.yao

  • Status changed from new to assigned
  • Owner set to garry.yao
  • Version set to 3.0

We should give priority, look at the affected version.

comment:2 Changed 5 years ago by garry.yao

  • Milestone changed from CKEditor 3.3 to CKEditor 3.4

comment:3 Changed 5 years ago by garry.yao

  • Keywords Review? added

comment:4 Changed 4 years ago by fredck

  • Milestone changed from CKEditor 3.4 to CKEditor 3.5

comment:5 Changed 4 years ago by fredck

#5818 has been marked as DUP.

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

comment:7 Changed 4 years ago by Saare

  • Status changed from review to review_passed

comment:8 Changed 4 years ago by garry.yao

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

Fixed with [5960].

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