Opened 8 years ago

Closed 8 years ago

Last modified 3 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 8 years ago.
3828.patch (2.7 KB) - added by Garry Yao 8 years ago.

Download all attachments as: .zip

Change History (12)

Changed 8 years ago by Damian

Attachment: TestList.doc added

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

Resolution: fixed
Status: assignedclosed

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

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

Keywords: Review? added

Changed 8 years ago by Garry Yao

Attachment: 3828.patch added

comment:7 Changed 8 years ago by Garry Yao

Milestone: CKEditor 3.1CKEditor 3.0.1

comment:8 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

comment:9 Changed 8 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 3 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