Implement Paste from Word dialog

[Garry Yao]

This bug cause comments tag content being inserted as codes when switch to 'source' mode after pasting MS-WORD(2003) documents which always contents IE-targeted conditional comments.

1. Open an editor instance with 'source' mode and paste the content of attachment 'msword-comments-doc' into 'wysiwyg' mode.
2. Switch to 'source' mode
   • Expected: 'source' mode contains correct content markups.
   • Actual: 'source' mode contains comment tag inner content.

Edit (2009-01-16): This is actually a missing feature.

[Garry Yao]

reproducing attachment

[Martin Kou]

Do you mean these things left in the beginning of the HTML output?

Normal 0  0 2  false false false               MicrosoftInternetExplorer4         /* Style Definitions */ table.MsoNormalTable {mso-style-name:"Table Normal"; mso-tstyle-rowband-size:0; mso-tstyle-colband-size:0; mso-style-noshow:yes; mso-style-parent:""; mso-padding-alt:0cm 5.4pt 0cm 5.4pt; mso-para-margin:0cm; mso-para-margin-bottom:.0001pt; mso-pagination:widow-orphan; font-size:10.0pt; font-family:"Times New Roman"; mso-ansi-language:#0400; mso-fareast-language:#0400; mso-bidi-language:#0400;}

These are actually fragments of <style> and <meta> contents pasted from MS Word. We have a "Paste from Word" dialog in v2 for cleaning up these things, but it isn't ported to v3, yet. So it's not a bug, but rather a missing feature.

[Frederico Caldeira Knabben]

• The cleanWord function must be a property of the dialog definition. In this way it's possible to easily replace it.

• The htmlToLoad string must also go into the dialog definition. It doesn't need to blow the core.

• Again... do not overuse CKEDITOR.tools.extend. The settings can be defined each one separately, like CKEDITOR.config.pasteFromWordIgnoreFontFace.

[Frederico Caldeira Knabben]

I didn't look at this patch in depth, but in the Trac preview, I can note we have an unneeded (function(){})(); holding the _source/plugins/pastefromword/dialogs/pastefromword.js code.

[Martin Kou]

Ah, yes, I forgot to remove the outer closure after removing the local variables.

[Martin Kou]

Fixed with [3083].