Opened 16 years ago

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

Download all attachments as: .zip

Change History (9)

comment:1 Changed 16 years ago by Wiktor Walc

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

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

Status: new → confirmed

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 16 years ago by Frederico Caldeira Knabben

Milestone: → CKEditor 3.5.1

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

Status: confirmed → pending

WFM on trunk.

comment:5 Changed 16 years ago by Garry Yao

Status: pending → confirmed

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 16 years ago by Garry Yao

Attachment: 6630.patch​ added

comment:6 Changed 16 years ago by Garry Yao

Owner: set to Garry Yao
Status: confirmed → review

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

Status: review → review_passed

comment:8 Changed 16 years ago by Garry Yao

Resolution: → fixed
Status: review_passed → closed

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