Opened 7 years ago

Closed 7 years ago

Last modified 4 years ago

#6330 closed Bug (fixed)

Roman list style are not pasted properly from Word

Reported by: Wiktor Walc Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.4.3
Component: Plugin : Paste from Word Version: 3.1
Keywords: IBM Cc:

Description (last modified by Wiktor Walc)

When pasting from Word, lists like:

i. foo
ii. bar

become

a. foo
b. bar

It is wrong, list style type should be preserved.

Reproduced in 3.4, worked fine in 3.0. I could not reproduce it in other versions due to #5399.

Confirmed on: Win 7 Professional N 64-bit EN + Office 2010 standard (en_office_standard_2010_x64_515548.exe)

Attachments (3)

ckeditor_pasting_bullets.png (22.2 KB) - added by Wiktor Walc 7 years ago.
bullets.doc (23.5 KB) - added by Wiktor Walc 7 years ago.
6330.patch (1.3 KB) - added by Garry Yao 7 years ago.

Download all attachments as: .zip

Change History (16)

Changed 7 years ago by Wiktor Walc

Changed 7 years ago by Wiktor Walc

Attachment: bullets.doc added

comment:1 Changed 7 years ago by Wiktor Walc

Status: newconfirmed

comment:2 Changed 7 years ago by Wiktor Walc

Component: GeneralCore : Pasting

comment:3 Changed 7 years ago by Wiktor Walc

Description: modified (diff)

comment:4 Changed 7 years ago by Garry Yao

Milestone: CKEditor 3.5
Owner: set to Garry Yao
Status: confirmedreview
Summary: Ordered lists are not pasted properly from WordRoman list style are not pasted properly from Word

Changed 7 years ago by Garry Yao

Attachment: 6330.patch added

comment:5 Changed 7 years ago by Garry Yao

Keywords: IBM added

comment:6 Changed 7 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.5CKeditor 3.4.3

comment:7 Changed 7 years ago by Frederico Caldeira Knabben

Keywords: IE added

comment:8 Changed 7 years ago by Frederico Caldeira Knabben

Status: reviewreview_failed

It works well for me with Firefox, even before the patch.

In IE, I'm always having the following output, even after patch (it makes no difference):

1. asasd 
  1. aasdsd 
  2. asd
2. asd 
  1. asd 
  2. asd 
    1. asdasd 
    2. asd 
      1. asdasd 
        1. asdasd
  3. asd

I'm using the document attached by wwalc for testing.

comment:9 Changed 7 years ago by Garry Yao

Status: review_failedreview
  1. Firefox illustration you see is a recent Firefox regression see #6662;
  2. For the patch to work need pasteFromWordRemoveStyles turned off, as list-style-type is actually a style.

comment:10 Changed 7 years ago by Garry Yao

Keywords: IE removed

comment:11 Changed 7 years ago by Frederico Caldeira Knabben

Status: reviewreview_passed

I'm accepting the patch because it has nothing to do with it, but I think we have something to rethink in the future. We, developers, know that list-style-type is related to styles, but I'm not sure that's the way end users understand it, so it should not depend on pasteFromWordRemoveStyles much probably.

comment:12 Changed 7 years ago by Garry Yao

Resolution: fixed
Status: review_passedclosed

Fixed with [6115], the issue mentioned by Fred will be tackled at #6635.

comment:13 Changed 4 years ago by Frederico Caldeira Knabben

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