Opened 7 years ago

Closed 7 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 7 years ago.
5595_2.patch (2.4 KB) - added by Garry Yao 7 years ago.

Download all attachments as: .zip

Change History (10)

Changed 7 years ago by Garry Yao

Attachment: 5595.patch added

comment:1 Changed 7 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 7 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.3CKEditor 3.4

comment:3 Changed 7 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.4CKEditor 3.5

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

Keywords: Confirmed removed
Status: reviewreview_failed

Tags with attributes should also be treated.

comment:5 Changed 7 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 7 years ago by Garry Yao

Attachment: 5595_2.patch added

comment:6 Changed 7 years ago by Garry Yao

Status: review_failedreview

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 [5953].

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