Opened 10 years ago

Closed 10 years ago

Last modified 5 years ago

#3828 closed New Feature (fixed)

Paste from Word - bullets not reproduced

Reported by: Damian Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.1
Component: Plugin : Paste from Word Version:
Keywords: IBM Confirmed Paste Review+ Cc:

Description

When using Paste from Word dialog to paste a bullet list with indented bullet items the list is not preserved correctly. It appears that that sub lists are actually created as separate lists.

Use attached sample Word document to reproduce.

Attachments (2)

TestList.doc (23.5 KB) - added by Damian 10 years ago.
3828.patch (2.7 KB) - added by Garry Yao 10 years ago.

Download all attachments as: .zip

Change History (12)

Changed 10 years ago by Damian

Attachment: TestList.doc added

comment:1 Changed 10 years ago by Garry Yao

Keywords: Discussion added
Type: BugNew Feature

Currently we don't support mso-list in our paste from word function, so it's a feature.

comment:2 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: Confirmed added; Discussion removed
Milestone: CKEditor 3.0CKEditor 3.1

We're not supposed to have changes to the paster from Words features. We have enhancements planned for the 3.1, and we could consider this one at that point.

comment:3 Changed 10 years ago by Garry Yao

Keywords: Paste added
Owner: set to Garry Yao
Status: newassigned

The list in the attachment doc is representing the following HTML( which is surprising, since we normally don't expect an output with correct html list tags ):

<ul style="margin-top: 0cm;" type="disc">
	<li class="MsoNormal" style=""><span lang="EN-IE">Sdfd</span></li>
	<ul style="margin-top: 0cm;" type="circle">
		<li class="MsoNormal" style=""><span lang="EN-IE">This line is indented bullets</span>
		</li>
		<ul style="margin-top: 0cm;" type="square">
			<li class="MsoNormal" style=""><span lang="EN-IE">And this line is indented again</span>
			</li>
		</ul>
	</ul>
</ul>

It would be helpful if you could give me some clue on how this doc is produced( was unable to create a list manually in MS-Word version 2003/2007 which output the same markup as it )? Or it's been generated( Convert from some other application format)?

In any case, we'll probably target cleanup to this as a more generic html cleanup( <ul>s should not be nested ) instead of a MS-Word only cleanup.

comment:4 Changed 10 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed

Changes are committed with [4312] and [4313] at pasting branch.

comment:5 Changed 10 years ago by Garry Yao

Resolution: fixed
Status: closedreopened

After a discussion, this particular fixing should be present as a core fixing at the parser level, makes it more generic.

comment:6 Changed 10 years ago by Garry Yao

Keywords: Review? added

Changed 10 years ago by Garry Yao

Attachment: 3828.patch added

comment:7 Changed 10 years ago by Garry Yao

Milestone: CKEditor 3.1CKEditor 3.0.1

comment:8 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

comment:9 Changed 10 years ago by Garry Yao

Resolution: fixed
Status: reopenedclosed

Fixing with [4320] at trunk and reverting invalid pasting branch changes with [4321].

comment:10 Changed 5 years ago by Frederico Caldeira Knabben

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