Opened 10 years ago

Closed 10 years ago

Last modified 7 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 10 years ago.
bullets.doc (23.5 KB) - added by Wiktor Walc 10 years ago.
6330.patch (1.3 KB) - added by Garry Yao 10 years ago.

Download all attachments as: .zip

Change History (16)

Changed 10 years ago by Wiktor Walc

Changed 10 years ago by Wiktor Walc

Attachment: bullets.doc added

comment:1 Changed 10 years ago by Wiktor Walc

Status: newconfirmed

comment:2 Changed 10 years ago by Wiktor Walc

Component: GeneralCore : Pasting

comment:3 Changed 10 years ago by Wiktor Walc

Description: modified (diff)

comment:4 Changed 10 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 10 years ago by Garry Yao

Attachment: 6330.patch added

comment:5 Changed 10 years ago by Garry Yao

Keywords: IBM added

comment:6 Changed 10 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.5CKeditor 3.4.3

comment:7 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: IE added

comment:8 Changed 10 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 10 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 10 years ago by Garry Yao

Keywords: IE removed

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

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