[FF3] simple copy & paste from Word document - extra code not stripped

[icedblind]

You can check this bug by yourself trying to copy and paste (CTRL+C/V) some text from a Microsoft Word document using first FF2 and after FF3 in Demo FCKeditor pages.

In FF3, viewing the source code of the copied text, you can see some extra information that in FF2 is stripped (meta tags, xml and style definitions):

<meta content="text/html; charset=utf-8" http-equiv="Content-Type" />
<meta content="Word.Document" name="ProgId" />
<meta content="Microsoft Word 11" name="Generator" />
<meta content="Microsoft Word 11" name="Originator" />
<link href="file:///[...]" rel="File-List" /><!--[if gte mso 9]><xml>
<w:WordDocument>
<w:View>Normal</w:View>
[...]
</xml><![endif]--><style type="text/css">
<!--
 /* Style Definitions */
[...]
</style>
<![endif]-->

[fredck]

You should use the "Paste from Word" button to have better results. The normal pasting will take the clipboard data as is, and we can see that the Firefox team have worked to make it "better" for their version 3.

[icedblind]

I'm sorry fredck, but that does not resolve the problem: if you try, also using the "Paste from Word" button you get quite the same result (ok, something more is stripped, but you don't get the same result as using FF2).

<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
<meta content="Word.Document" name="ProgId">
<meta content="Microsoft Word 11" name="Generator">
<meta content="Microsoft Word 11" name="Originator">
<link href="file:///[...]" rel="File-List" /><!--[if gte mso 9]><xml>
  Normal
  0
  false
  false
  false
  [...]
  MicrosoftInternetExplorer4
</xml><![endif]--><!--[if gte mso 9]><![endif]--><style type="text/css">
<!--
 /* Style Definitions */
[...]
</style><!--[if gte mso 10]>
<style>
 /* Style Definitions */
 [...]
</style>
<![endif]-->

[icedblind]

fredck, i've taken a look at fckeditor/editor/dialog/fck_paste.html code, inside CleanWord function and i propose to modify some code - if you all don't see any problem, i'm not a smart js programmer''

fckeditor/editor/dialog/fck_paste.html - line 248

actual

html = html.replace(/<\!--.*?-->/g, '' ) ;

new lines

html = html.replace( /<w:[^>]*>(.*?)<\/w:[^>]*>/gi, '' ) ;
html = html.replace( /<meta[^>]*>/gi, '' ) ;
html = html.replace( /<link[^>]*>/gi, '' ) ;
html = html.replace( /<style[^>]*>([\w|\W|\n]*?)<\/style>/gim, '' ) ;
html = html.replace( /<\!--([\w|\W|\n]*?)-->/gm, '' ) ;

This at least strips away the lines that i've reported in FF3, and also extends to multiplelines the comment removal using the "paste from word" button.

[fredck]

Your suggestion makes sense icedblind. Thanks for it.

The provided regexes are not the definitive though. The "catch all" for JavaScript is /[\s\S]*/. Also, <meta> and <link> could be caught on a single regex /(?:meta|link)/.

We are aware that the Word cleanup procedure is to be refined with time. We can't catch all cases in a set of tests, so additions like yours would just make it better.

I had previously invalidated the ticket because it was making reference to the plain pasting operation, which is out of our control. Now, enhancements to Paste from Word are definitely acceptable.

[icedblind]

I think it's great. Thank-you fredck!

[alfonsoml]

#2272 does already suggest some changes to the regexp cleaning.

[fredck]

Actually, the suggestions at #2272 make a lot of sense. Closing this one as DUP.