Opened 13 years ago

Closed 12 years ago

#7308 closed Bug (fixed)

Spacers being added after tables using Firefox 3.6.15

Reported by: Matt Hartman 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 (1)

table.png (37.5 KB) - added by Jakub Ś 13 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 13 years ago by Garry Yao

Status: newpending

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

comment:2 Changed 13 years ago by Jakub Ś

Keywords: Firefox3 added
Status: pendingconfirmed
Version: 3.5.23.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 13 years ago by Garry Yao

Status: confirmedpending

WFM with FF4/Win7.

comment:4 Changed 13 years ago by Jakub Ś

Checked with latest CKEditor. Still occurs.

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

Changed 13 years ago by Jakub Ś

Attachment: table.png added

comment:5 Changed 13 years ago by Jakub Ś

Status: pendingconfirmed

comment:6 Changed 12 years ago by Jakub Ś

Resolution: fixed
Status: confirmedclosed

No longer reproducible in latest CKEditor 3.6.4.

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