Opened 7 years ago

Last modified 7 years ago

#8481 confirmed Bug

switching between source and wysiwyg introduces gratuitous white space

Reported by: Jeff Finucane Owned by:
Priority: Normal Milestone:
Component: General Version: 3.5.3
Keywords: Cc:

Description

Certain html snippets introduce extra white space on each switch between source and wysiwyg.

In 6.0.472.63 (59945) Built on Debian 6.0.2, running on Debian 6.0.3, Iceweasel/3.5.16 (like Firefox/3.5.16), and recent versions of Safari the attached snippet exhibits the problem in at least 3.6.2 and the nightly build as of 2011-10-27.

The behavior is somewhat different (less desireable?) with fullPage : true than without.

In both cases each round trip wysiwyg->source->wysiwyg introduces a <p>&nbsp;</p> fragment.

Attachments (1)

thefile (81 bytes) - added by Jeff Finucane 7 years ago.
html fragment causing undesired behavior

Download all attachments as: .zip

Change History (5)

Changed 7 years ago by Jeff Finucane

Attachment: thefile added

html fragment causing undesired behavior

comment:1 Changed 7 years ago by Jakub Ś

Status: newconfirmed
Version: 3.6.23.5.3

Although html is invalid this doesn't explain why every time when you switch to source and back new paragraph is created.

I have been able to reproduce it from CKEditor 3.5.3 rev [6629] which was a fix for [6561] which, based on example provided, would change the first line with invalid HTML into empty p.

This issue is reproducible in all browsers.

comment:2 Changed 7 years ago by Jakub Ś

Other tickets where this revision is mentioned: #8189, #8551, #9186, #9006.

Last edited 7 years ago by Jakub Ś (previous) (diff)

comment:3 Changed 7 years ago by Jakub Ś

<p style="display: none;">
&nbsp;</p>
<link href="assets/images/HomePageArt/2012-08/august-08-02-bg.css" rel="stylesheet" type="text/css" />
<link href="assets/images/HomePageArt/home-page.css" rel="stylesheet" type="text/css" />
<div id="slides">
<div id="slides-overlay" style="width:100px;height:51px;background:none;">
<ul class="pagination">
<li>
<a href="#0" title="Back to School Sale">&nbsp;</a></li>
<li>
<a href="#1" title="End of Summer Sale">&nbsp;</a></li>
<li>
<a href="#3" title="Fee gift with Purchase">&nbsp;</a></li>
</ul>
</div>
</div>

Another piece of invalid code reported on our support channel.
Till CKEditor 3.5.3 rev [6629] such invalid links were ignored which was more user friendly and was making editor more flexible IMHO.

comment:4 Changed 7 years ago by Jean Beha

We are experiencing this problem with some of our customers that have an invalid link tag included in their partial page source (Please see sample we sent in previous comment). We recently upgraded from FCKEditor to CKEditor (3.4.6), and its a big disatisfier to our customers to have the white space entered into fields that previously did not display this behavior. Thanks...

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