Opened 13 years ago

Closed 13 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 13 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 13 years ago by Wiktor Walc

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

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

Milestone: CKEditor 3.5.1

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

Status: confirmedpending

WFM on trunk.

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

Attachment: 6630.patch added

comment:6 Changed 13 years ago by Garry Yao

Owner: set to Garry Yao
Status: confirmedreview

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

Status: reviewreview_passed

comment:8 Changed 13 years ago by Garry Yao

Resolution: fixed
Status: review_passedclosed

Fixed with [6321].

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy