Opened 5 years ago

Closed 5 years ago

#11660 closed Bug (fixed)

IE8 table content is lost when there is some extra markup inside a table

Reported by: Wiktor Walc Owned by: Piotrek Koszuliński
Priority: Normal Milestone: CKEditor 4.3.4
Component: General Version: 3.6.3
Keywords: Cc:

Description

Take the following code and paste in source mode in IE8:

<table border="1" cellpadding="1" cellspacing="1" style="width:500px">
		<tr>
			<td>1</td>
			<td>2</td>
Issue1
		</tr>
Issue2
		<tr>
			<td>3</td>
			<td>4</td>
		</tr>
		<tr>
			<td>5</td>
			<td>6</td>
		</tr>
</table>

The result is:

<table border="1" cellpadding="1" cellspacing="1" style="width: 500px">
	<tbody>
		<tr>
			<td>
				1</td>
			<td>
				2</td>
			<td>
				Issue1</td>
		</tr>
	</tbody>
</table>

The expected result is having strings "Issue1" and "Issue2" moved at the top of the table and the table itself left intact.

Note 1: plain contenteditable behaves correctly

Note 2: this is a misbehaviour of CKEditor, introduced in CKEditor 3.6.3, exactly in this commit: [7411]. See samples for 7410 vs 7411

Change History (5)

comment:1 Changed 5 years ago by Wiktor Walc

Status: newconfirmed

comment:2 Changed 5 years ago by Piotrek Koszuliński

Owner: set to Piotrek Koszuliński
Status: confirmedreview

Pushed branch:t/11660 on dev and tests.

comment:3 Changed 5 years ago by Piotrek Koszuliński

Milestone: CKEditor 4.3.4

comment:4 Changed 5 years ago by Frederico Caldeira Knabben

Status: reviewreview_passed

Well done!

comment:5 Changed 5 years ago by Piotrek Koszuliński

Resolution: fixed
Status: review_passedclosed

Fixed on master with git:5cd42b7 on dev and 6cf8502 on tests.

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