Opened 7 years ago

Closed 7 years ago

#6630 closed Bug (fixed)

Extra empty line inside of formatted element

Reported by: Wiktor Walc Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.5.1
Component: General Version: 3.4
Keywords: Cc:

Description

In CKEditor 3.3.1 creating and empty formatted element resulted in:

<pre>
</pre>

In CKEditor 3.4 we had:

<pre>
<span style="display: none;"> </span>
</pre>

now (3.4.2 SVN) we have:

<pre>

</pre>

we should get back to the right version.

Attachments (1)

6630.patch (2.1 KB) - added by Garry Yao 7 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 7 years ago by Wiktor Walc

(results from Firefox 3.6.12 / win 7, safe mode)

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

Status: newconfirmed

With IE8 the result is different:
3.4.1:

<pre>
<span style="display: none"> </span></pre>

3.4.2 SVN:

<pre>
</pre>

The last result is OK.

comment:3 Changed 7 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.5.1

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

Status: confirmedpending

WFM on trunk.

comment:5 Changed 7 years ago by Garry Yao

Status: pendingconfirmed

Confirmed, right now empty preformatted block contains whitespaces that depends on browser, let's make necessary normalization, reverting to the result we have in 3.1.

Changed 7 years ago by Garry Yao

Attachment: 6630.patch added

comment:6 Changed 7 years ago by Garry Yao

Owner: set to Garry Yao
Status: confirmedreview

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

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