Opened 10 years ago

Closed 10 years ago

#5595 closed Bug (fixed)

Leading line-breaks lost in <pre>

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

Description

Reproducing Procedures

  1. Open any of the sample page and load the editor with the following content;
    <pre>
    
    Two leading breaks, one visible only.</pre>
    
    • Actual Result: The line-break is invisible in wysiwyg mode.

Attachments (2)

5595.patch (1.1 KB) - added by Garry Yao 10 years ago.
5595_2.patch (2.4 KB) - added by Garry Yao 10 years ago.

Download all attachments as: .zip

Change History (10)

Changed 10 years ago by Garry Yao

Attachment: 5595.patch added

comment:1 Changed 10 years ago by Garry Yao

Keywords: Review? added
Owner: set to Garry Yao
Status: newassigned

Design TC updated:
run OR view source.

comment:2 Changed 10 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.3CKEditor 3.4

comment:3 Changed 10 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.4CKEditor 3.5

comment:4 Changed 10 years ago by Sa'ar Zac Elias

Keywords: Confirmed removed
Status: reviewreview_failed

Tags with attributes should also be treated.

comment:5 Changed 10 years ago by Sa'ar Zac Elias

And also.. if the breakAfterOpen option for the pre tag was false before, the result would not be the same.

Changed 10 years ago by Garry Yao

Attachment: 5595_2.patch added

comment:6 Changed 10 years ago by Garry Yao

Status: review_failedreview

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

Status: reviewreview_passed

comment:8 Changed 10 years ago by Garry Yao

Resolution: fixed
Status: review_passedclosed

Fixed with [5953].

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