Ticket #7308 (closed Bug: fixed)

Opened 3 years ago

Last modified 21 months ago

Spacers being added after tables using Firefox 3.6.15

Reported by: fourmat Owned by:
Priority: Normal Milestone:
Component: General Version: 3.0
Keywords: Firefox3 Cc:

Description

Using Firefox 3.6.15 if I edit a table of the following configuration:

<table align="center" border="0" cellpadding="4" cellspacing="0" width="500">
	<tbody>
		<tr bgcolor="#ffffff">
			<td nowrap="nowrap" style="font-family: Tahoma,Arial,Helvetica,sans-serif; font-size: 11px;" width="30%">
				<span style="font-family: Arial;"><strong>EMI Item #</strong></span></td>
			<td style="font-family: Tahoma,Arial,Helvetica,sans-serif; font-size: 11px;" width="1%">
				&nbsp;</td>
			<td style="font-family: Tahoma,Arial,Helvetica,sans-serif; font-size: 11px;" width="69%">
				<span style="font-family: Arial;">EMI-48100</span></td>
		</tr>
		<tr bgcolor="#f2f2f2">
			<td nowrap="nowrap" style="font-family: Tahoma,Arial,Helvetica,sans-serif; font-size: 11px;" width="30%">
				<span style="font-family: Arial;"><span style="color: rgb(0, 0, 0);"><strong>Item Type</strong></span></span></td>
			<td style="font-family: Tahoma,Arial,Helvetica,sans-serif; font-size: 11px;" width="1%">
				<span style="font-family: Arial;"><span style="color: rgb(0, 0, 0);">&nbsp;</span></span></td>
			<td style="font-family: Tahoma,Arial,Helvetica,sans-serif; font-size: 11px;" width="69%">
				<span style="font-family: Arial;"><span style="color: rgb(0, 0, 0);">Tape</span></span></td>
		</tr>
		<tr bgcolor="#ffffff">
			<td nowrap="nowrap" style="font-family: Tahoma,Arial,Helvetica,sans-serif; font-size: 11px;" width="30%">
				<span style="font-family: Arial;"><span style="color: rgb(0, 0, 0);"><strong>Product Type</strong></span></span></td>
			<td style="font-family: Tahoma,Arial,Helvetica,sans-serif; font-size: 11px;" width="1%">
				<spacer height="1" type="block" width="1"></spacer></td>
			<td style="font-family: Tahoma,Arial,Helvetica,sans-serif; font-size: 11px;" width="69%">
				<span style="font-family: Arial;"><span style="color: rgb(0, 0, 0);">Anti-Skid Tape</span></span></td>
		</tr>
		<tr bgcolor="#f2f2f2">
			<td nowrap="nowrap" style="font-family: Tahoma,Arial,Helvetica,sans-serif; font-size: 11px;" width="30%">
				<span style="font-family: Arial;"><span style="color: rgb(0, 0, 0);"><b>Color</b></span></span></td>
			<td style="font-family: Tahoma,Arial,Helvetica,sans-serif; font-size: 11px;" width="1%">
				<spacer height="1" type="block" width="1"></spacer></td>
			<td style="font-family: Tahoma,Arial,Helvetica,sans-serif; font-size: 11px;" width="69%">
				<span style="font-family: Arial;"><span style="color: rgb(0, 0, 0);">Black</span></span></td>
		</tr>
		<tr>
			<td nowrap="nowrap" style="font-family: Tahoma,Arial,Helvetica,sans-serif; font-size: 11px;" width="30%">
				&nbsp;</td>
			<td style="font-family: Tahoma,Arial,Helvetica,sans-serif; font-size: 11px;" width="1%">
				&nbsp;</td>
			<td style="font-family: Tahoma,Arial,Helvetica,sans-serif; font-size: 11px;" width="69%">
				&nbsp;</td>
		</tr>
		<tr>
			<td nowrap="nowrap" style="font-family: Tahoma,Arial,Helvetica,sans-serif; font-size: 11px;" width="30%">
				&nbsp;</td>
			<td style="font-family: Tahoma,Arial,Helvetica,sans-serif; font-size: 11px;" width="1%">
				&nbsp;</td>
			<td style="font-family: Tahoma,Arial,Helvetica,sans-serif; font-size: 11px;" width="69%">
				&nbsp;</td>
		</tr>
		<tr>
			<td nowrap="nowrap" style="font-family: Tahoma,Arial,Helvetica,sans-serif; font-size: 11px;" width="30%">
				&nbsp;</td>
			<td style="font-family: Tahoma,Arial,Helvetica,sans-serif; font-size: 11px;" width="1%">
				&nbsp;</td>
			<td style="font-family: Tahoma,Arial,Helvetica,sans-serif; font-size: 11px;" width="69%">
				&nbsp;</td>
		</tr>
		<tr>
			<td nowrap="nowrap" style="font-family: Tahoma,Arial,Helvetica,sans-serif; font-size: 11px;" width="30%">
				&nbsp;</td>
			<td style="font-family: Tahoma,Arial,Helvetica,sans-serif; font-size: 11px;" width="1%">
				&nbsp;</td>
			<td style="font-family: Tahoma,Arial,Helvetica,sans-serif; font-size: 11px;" width="69%">
				&nbsp;</td>
		</tr>
	</tbody>
</table>

ckeditor inserts spacer tags after the table each time the code view is toggled, or on loading on the editor. Each time the code view is toggled more tags are added.

<p>
	<spacer height="1" type="block" width="1"><spacer height="1" type="block" width="1">
	<p>
		<spacer height="1" type="block" width="1"><spacer height="1" type="block" width="1"></spacer></spacer></p>
	<spacer height="1" type="block" width="1"><spacer height="1" type="block" width="1">
	<p>
		<spacer height="1" type="block" width="1"><spacer height="1" type="block" width="1"> </spacer></spacer></p>
	<spacer height="1" type="block" width="1"><spacer height="1" type="block" width="1"><spacer height="1" type="block" width="1"><spacer height="1" type="block" width="1">
	<p>
		&nbsp;</p>
	</spacer></spacer></spacer></spacer></spacer></spacer></spacer></spacer></p>

I can only reproduce this in Firefox, and it does not happen with every table. I think is has something to do with the number of rows. If I remove rows I can get it to stop. Also, if I remove the empty column it will not add the spacers at the bottom.

This can be duplicated in the Demo by pasting the code and toggling the Code View.

Attachments

table.png (37.5 KB) - added by j.swiderski 3 years ago.

Change History

comment:1 Changed 3 years ago by garry.yao

  • Status changed from new to pending

WFM on latest build, could you provide a sample page for reproducing?

comment:2 Changed 3 years ago by j.swiderski

  • Status changed from pending to confirmed
  • Keywords Firefox3 added
  • Version changed from 3.5.2 to 3.0

Steps to reproduce:

  1. Open replacebycode sample
  2. Paste the table in source mode
  3. Switch to WYSIWYG
  4. Open "Table Properties" dialog and close it by clicking ok.
  5. Now toggle few times between source and WYSIWYG

You will see that spacer tags are added at the bottom of source code.

This is true for Firefox 3.6 on Win XP and Win7.

comment:3 Changed 3 years ago by garry.yao

  • Status changed from confirmed to pending

WFM with FF4/Win7.

comment:4 Changed 3 years ago by j.swiderski

Checked with latest CKEditor. Still occurs.

Have you followed my steps? I have managed to reproduce in the first try (Please see attachment).

Changed 3 years ago by j.swiderski

comment:5 Changed 3 years ago by j.swiderski

  • Status changed from pending to confirmed

comment:6 Changed 21 months ago by j.swiderski

  • Status changed from confirmed to closed
  • Resolution set to fixed

No longer reproducible in latest CKEditor 3.6.4.

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