Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#11237 closed Bug (fixed)

Paste table from Microsoft Office Word - lost value of border attribute

Reported by: peter Owned by: Piotrek Koszuliński
Priority: Normal Milestone: CKEditor 4.3.1
Component: Plugin : Paste from Word Version: 4.3
Keywords: IBM Firefox Webkit Blink Cc: Teresa Monahan, Satya Minnekanti, Irina

Description

Open Ajax sample.

  1. Create table in MSWord, select all and copy.
  2. Paste into the editor body or through Paste Dialog.
  3. Open Table Properties dialog and check for Border size value.

Expected result: Border size should be set to 1

Actual result: Border size field is empty.

It's happens since 4.3 in FF and Chrome, works fine in IE.

see the attached MSWord with sample table.

Attachments (2)

borderBug.doc (26.5 KB) - added by peter 11 years ago.
table_html.txt (3.7 KB) - added by Jakub Ś 11 years ago.

Download all attachments as: .zip

Change History (12)

Changed 11 years ago by peter

Attachment: borderBug.doc added

comment:1 Changed 11 years ago by Jakub Ś

Keywords: Firefox Webkit Blink added
Status: newconfirmed

Problem can be reproduced from CKEditor 4.3 in FF, Webkit and Blink.

Using below three options helps to get border in table

allowedContent : true,
pasteFromWordRemoveFontStyles : false,
pasteFromWordRemoveStyles : false

but resulting HTML is rather not expected:

<table border="" cellpadding="0" cellspacing="0" style="border-collapse:collapse;border:none;">
	<tbody>
		<tr>
			<td style="width:213px;border:solid windowtext 1.0pt;padding:0cm 5.4pt 0cm 5.4pt;vertical-align:top;">
			<p style="margin-bottom:0cm;margin-bottom:.0001pt;line-height:normal;">Cell 1</p>
			</td>
			<td style="width:213px;border:solid windowtext 1.0pt;border-left:none;padding:0cm 5.4pt 0cm 5.4pt;vertical-align:top;">
...

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

cc

comment:3 Changed 11 years ago by Jakub Ś

git:3b806f7 is the first bad commit

comment:4 Changed 11 years ago by Piotr Jasiun

Owner: set to Piotr Jasiun
Status: confirmedassigned

Changed 11 years ago by Jakub Ś

Attachment: table_html.txt added

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

Owner: changed from Piotr Jasiun to Piotrek Koszuliński

comment:6 Changed 11 years ago by Piotrek Koszuliński

Status: assignedreview

Pushed t/11237 on dev and tests.

comment:7 Changed 11 years ago by Jakub Ś

Status: reviewreview_passed

Works fine. Borders are now recognized correctly.

comment:8 Changed 11 years ago by Piotrek Koszuliński

Milestone: CKEditor 4.3.1

comment:9 Changed 11 years ago by Piotrek Koszuliński

Resolution: fixed
Status: review_passedclosed

Fixed on master with git:37761cf on dev and b7af0f2 on tests.

comment:10 Changed 11 years ago by Frederico Caldeira Knabben

Component: GeneralPlugin : Paste from Word
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