817986 - Pasting only HTML table into composition editor fails (but works for pasting table with surrounding content)

Status: UNCONFIRMED

(Thunderbird :: Message Compose Window, defect)

Description

[Dennis Jakobsen]

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:20.0) Gecko/20121203 Firefox/20.0
Build ID: 20121203030801

Steps to reproduce:

1. mark and copied a table from Firefox
2. tried to paste it in thunderbird compose window


Actual results:

nothing (works with "paste without formatting")


Expected results:

it used to paste tables just fine

Comment 1

[Thomas D. (:thomas8)]

worksforme on WinXP/TB17, with this testcase:
http://www.w3schools.com/html/html_tables.asp

Reporter, can you copy that table from w3schools and see if that works?

Can you provide us with the URL to your table which didn't copy?
Without a testcase, there's nothing we can do here.
Copying rich text is not generally broken.

Comment 2

[Thomas D. (:thomas8)]

Dennis, feel free to reopen when you have more details, especially answer to comment 1. Until then, this is incomplete.

Comment 3

[Dennis Jakobsen]

I'd be happy to! Here's the error console output of trying to paste an html table:

Timestamp: 26-12-2012 12:26:09
Error: An error occurred executing the cmd_paste command: [Exception... "Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIController.doCommand]"  nsresult: "0x80004005 (NS_ERROR_FAILURE)"  location: "JS frame :: chrome://global/content/globalOverlay.js :: goDoCommand :: line 91"  data: no]
Source File: chrome://global/content/globalOverlay.js
Line: 95

the tables paste into notepad and notepad++ just fine.

Comment 4

[Dennis Jakobsen]

Disabled all extentions and addons, from Paste As Quotation aswell:

Timestamp: 26-12-2012 12:31:16
Error: An error occurred executing the cmd_pasteQuote command: [Exception... "Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIController.doCommand]"  nsresult: "0x80004005 (NS_ERROR_FAILURE)"  location: "JS frame :: chrome://global/content/globalOverlay.js :: goDoCommand :: line 91"  data: no]
Source File: chrome://global/content/globalOverlay.js
Line: 95

Paste without formatting still works.

Comment 5

[Thomas D. (:thomas8)]

Dennis, thanks for rapid reply. Error messages are very helpful.
However, we also need your testcase tables to try and reproduce your error, plus answers to comment 1, because we can't do anything about this problem as long as we don't see it.

Pls do the following:

1a) let us know an URL to those tables which cause errors of comment 3 and comment 4 - OR - 
1b) attach tables of comment 3 and comment 4 to this bug ("Add an attachment" button above)

AND

2) Copy and paste the first table from testcase of comment 1 (mouse-select from "Apples" down to "10%"), then report if that works or not, and any error messages:
http://www.w3schools.com/html/html_tables.asp

Thanks.

Comment 6

[Dennis Jakobsen]

Attachment: testcase1.zip: isolated html table, that cannot be pasted into compose window

Comment 7

[Dennis Jakobsen]

if i remove the (invalid html) span around the table it works..  but creating a 'clean' html file with just a span and a table inside is not an issue. so probably a combination of what's in the table and the fact that it's put inside the span. anyways, gl fixing it :)

Comment 8

[Thomas D. (:thomas8)]

Attachment: Testcase1.maff: isolated html table that cannot be pasted into compose window (use mozilla-archive-format addon to open, or rename to zip and extract to open)

Here's testcase 1 from attachment 695737 [details] in a more accessible format.
Use Mozilla Archive Format Addon [1] to open testcase1.maff directly in Firefox, or rename .maff file to .zip file, then extract and open contained html file set.

[1] https://addons.mozilla.org/en-us/firefox/addon/mozilla-archive-format/

Comment 9

[Thomas D. (:thomas8)]

Attachment: Testcase1.maff: isolated html table that cannot be pasted into compose window (use mozilla-archive-format addon to open, or rename to zip and extract to open)

[Slightly revised maff file (replaces comment 8).]

Here's testcase 1 from attachment 695737 [details] [©] [details] in a more accessible format.
Use Mozilla Archive Format Addon [1] to open testcase1.maff directly in Firefox, or rename .maff file to .zip file, then extract and open contained html file set.

[1] https://addons.mozilla.org/en-us/firefox/addon/mozilla-archive-format/

Comment 10

[Thomas D. (:thomas8)]

Attachment: Testcase1.maff: isolated html table that cannot be pasted into compose window (use mozilla-archive-format addon to open, or rename to zip and extract to open)

Grrr, too tired. Ignore comment 8 and 9 and their attachments.

[Slightly revised maff file]

Here's testcase 1 from attachment 695737 [details] [©] [details] [©] [details] in a more accessible format.
Use Mozilla Archive Format Addon [1] to open testcase1.maff directly in Firefox, or rename .maff file to .zip file, then extract and open contained html file set.

[1] https://addons.mozilla.org/en-us/firefox/addon/mozilla-archive-format/

Comment 11

[Thomas D. (:thomas8)]

Attachment: Testcase1.eml (saved draft): Testcase 1 Table copied into new HTML composition (generally works, some styles lost)

STR

1 Open testcase 1 from attachment 695737 [details] (zip) or attachment 695885 [details] (maff)
2 Select html file content with Ctrl+A, then Ctrl+C to copy
3 Start new HTML composition
4 Paste Table into new composition
5 save as draft

Actual result = Expected result
-> see this attachment
-> copying basically worksforme on TB17/WinXP (as opposed to comment 6 which claims it cannot be pasted into composition)
- some flaws in terms of css styling, but that's off-topic here

Comment 12

[Thomas D. (:thomas8)]

I see this bug now (and I think I've seen a similar bug before)

testcase 1 from attachment 695737 [details] [©] [details] (zip) or attachment 695885 [details] [©] [details] (maff) fails as reported in comment 0 when you select only the table:

(In reply to Thomas D. from comment #11)

> STR
> 
> 1 Open testcase 1 from attachment 695737 [details] (zip) or attachment
> 695885 [©] [details] (maff)
> 2 Select html table only:
start mouse selection from upper left corner of table, 1cm to the left of insert icon, then drag to select down to lower-right corner of table so that only the table is selected
> then Ctrl+C to copy
> 3 Start new HTML composition
> 4 Paste Table into new composition
> 
> Actual result

nothing (composition remains blank, with blinking cursor)

> Expected result

should paste correctly into composition.

Comment 13

[Thomas D. (:thomas8)]

Bug confirmed for WinXP/TB17, per comment 12.

Comment 14

[Thomas D. (:thomas8)]

Similar to Bug 770895.
Use Free Clipboard Viewer (1) to inspect clipboard contents.
(1) http://www.freeclipboardviewer.com

Comment 15

[Thomas D. (:thomas8)]

Attachment: Failing clipboard content from Bug 817986 Testcase1 (Table in HTML Format flavor from clipboard).txt

Here's the failing clipboard content (the selected table from testcase 1, attachment 695737 [details]), inspected with clipboard viewer per comment 8.

Any ideas why it fails to paste?

Comment 16

[realgrouchy]

I added some comments in Bug 649913, which appears to be a possibly related bug but describing a different problem. I've discovered two other problems that are related to pasting tables which I suspect are more closely related to this bug, which might help wean out test cases. I am running TB24.4.0 on OSX10.8.5 (admittedly, with a lot of add-ons)

UNDO PASTE:
A1: Open a new HTML e-mail (with a blank line of Body Text followed by a preformatted signature)
A2: Copy a row of the testcase table from Firefox (http://www.w3schools.com/html/html_tables.asp) and paste into the first line.
A3: Type CMD+Z to undo
Result: the table is removed, as is the line of body text, putting the signature in the first line. Adding a carriage return gives a line of preformatted text instead of body text.
Note: I think this problem does not occur if instead of copying from Firefox you instead copy a previously pasted copy from another TB compose window.


PASTE TABLES COPIED FROM START OF EMAIL:
B1: Open a new HTML e-mail (with a blank line of Body Text followed by a preformatted signature)
B2: Add a line break so there are two lines of Body Text 
B2: Copy a row of the testcase table from Firefox (http://www.w3schools.com/html/html_tables.asp) and paste into the first line.
B3: Instead of selecting just the table (by double-clicking "Jill" and dragging the cursor over to the cell containing "50" so that the three cells are highlighted), put the cursor on Jill and drag down, selecting the text in the cells.
B4: Copy and attempt to paste into a second new e-mail
Result: unable to paste. (Curously, though, it will paste the text into the "to" field.
Note: If there is a line of text above the table, it is possible to select from the text above the table to the text below the table and successfully copy/paste. Only the text of the table appears selected but the entire table moves. This actually sounds a lot like Bug 649913