Opened 5 years ago

Closed 5 years ago

Last modified 5 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 5 years ago.
table_html.txt (3.7 KB) - added by Jakub Ś 5 years ago.

Download all attachments as: .zip

Change History (12)

Changed 5 years ago by peter

Attachment: borderBug.doc added

comment:1 Changed 5 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 5 years ago by Piotrek Koszuliński

cc

comment:3 Changed 5 years ago by Jakub Ś

git:3b806f7 is the first bad commit

comment:4 Changed 5 years ago by Piotr Jasiun

Owner: set to Piotr Jasiun
Status: confirmedassigned

Changed 5 years ago by Jakub Ś

Attachment: table_html.txt added

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

Owner: changed from Piotr Jasiun to Piotrek Koszuliński

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

Status: assignedreview

Pushed t/11237 on dev and tests.

comment:7 Changed 5 years ago by Jakub Ś

Status: reviewreview_passed

Works fine. Borders are now recognized correctly.

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

Milestone: CKEditor 4.3.1

comment:9 Changed 5 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 5 years ago by Frederico Caldeira Knabben

Component: GeneralPlugin : Paste from Word
Note: See TracTickets for help on using tickets.
© 2003 – 2017 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy