Opened 4 years ago

Closed 4 years ago

Last modified 3 years ago

#1486271 closed Bugs (fixed)

HTML comments are shown when text is pasted from Office-like app

Reported by: nukeador Owned by:
Priority: 5 Milestone: 0.4-beta
Component: Other Version: 0.3.1
Severity: normal Keywords:
Cc:

Description

When you copy and pasted text from Office-like app to an HTML message and then you send the mail, the result is that hidden comments are shown in the message body.

Example of the mail received:

<!-- @page { margin: 2cm } P { margin-bottom: 0.21cm } A:link { so-language: zxx } -->
Body text goes here.

Attachments (2)

HTML test message.eml (1.6 KB) - added by nukeador 4 years ago.
add_paste_plugin_cleanup_word.patch (1.1 KB) - added by nukeador 4 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 4 years ago by alec

  • Milestone changed from later to 0.4-beta

please attach sample message.

Changed 4 years ago by nukeador

comment:2 Changed 4 years ago by nukeador

Here you have an example, as you can see some comments are added to message body, and it's even worse if it's pasted from MS Office (tons of comments are added).

comment:3 follow-up: Changed 4 years ago by alec

http://tinymce.moxiecode.com/punbb/viewtopic.php?id=12405 I'm not a MSOffice user, so please try this paste_auto_cleanup_on_paste:true.

comment:4 in reply to: ↑ 3 Changed 4 years ago by nukeador

Replying to alec:

http://tinymce.moxiecode.com/punbb/viewtopic.php?id=12405 I'm not a MSOffice user, so please try this paste_auto_cleanup_on_paste:true.

Doing that on /program/js/editor.js does not solve the problem (you can test it using another Office-like app like OpenOffice?, KOffice...), neither loading the paste plugin as suggested in tinymce forums.

That forum topic also points to this patch from another editor:

http://dev.fckeditor.net/ticket/2272
http://dev.fckeditor.net/changeset/2174

I don't know if it's something that should be patched by tinymce or roundcube, but it's a very common for average users to c&p text from their Office apps.

comment:5 Changed 4 years ago by alec

I've made a test. Adding "paste" to the list of plugins fixes issue. Tested with OpenOffice?.

comment:6 Changed 4 years ago by nukeador

You are right, I added the paste plugin in the worng line and the plugin wasn't loaded. I'm attaching the path.

Changed 4 years ago by nukeador

comment:7 Changed 4 years ago by alec

  • Resolution set to fixed
  • Status changed from new to closed

Fixed in [18153efe].

Note: See TracTickets for help on using tickets.