Closed
Bug 1427265
Opened 6 years ago
Closed 6 years ago
No icon shown for message dragged to attachment bucket due to missing type indication
Categories
(Thunderbird :: Message Compose Window, defect)
Thunderbird
Message Compose Window
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 59.0
People
(Reporter: jorgk-bmo, Assigned: jorgk-bmo)
Details
Attachments
(1 file)
1.66 KB,
patch
|
aceman
:
review+
|
Details | Diff | Splinter Review |
This old bug has become apparent now that space is reserved for items in the attachment bucket. When dragging a message to the attachment bucket, no icon is shown. This can be fixed manually in the DOM Inspector by appending contentType=message/rfc822 to the icon src. Note that icons are shown in the attachment area of sent messages or saved drafts. Attachment bucked icon src: moz-icon:mailbox-message://.../Inbox#15018 moz-icon:imap-message://.../INBOX#7277 Attachment area icon src for sent/draft messages: moz-icon://....eml?size=16&contentType=message/rfc822 Maybe fixable in the drop handler in MsgComposeCommands.js envelopeDragObserver.onDrop() for "text/x-moz-message" although I don't see how the icon is set there.
Assignee | ||
Comment 1•6 years ago
|
||
Any idea where that icon src is set (I haven't looked at all).
Flags: needinfo?(acelists)
Assignee | ||
Comment 2•6 years ago
|
||
Oops, false alarm. Adding contentType=message/rfc822 does nothing. The icon comes when adding .eml somewhere in the src.
Summary: No icon shown for message dragged to attachment bucket due to missing contentType=message/rfc822 → No icon shown for message dragged to attachment bucket due to missing type indication
Assignee | ||
Comment 4•6 years ago
|
||
Testing for *-message URLs is ugly, but done elsewhere in the file.
Comment on attachment 8939036 [details] [diff] [review] 1427265-message-icon.patch Review of attachment 8939036 [details] [diff] [review]: ----------------------------------------------------------------- Thanks, works for me. I get multiple errors while dragging the message into attachments (on linux), like this: 00:11:04.287 NS_ERROR_FAILURE: Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsITransferable.getTransferData] 1 msgMail3PaneWindow.js:1375 getFlavorData chrome://messenger/content/msgMail3PaneWindow.js:1375:5 But that probably isn't related.
Attachment #8939036 -
Flags: review?(acelists) → review+
Assignee | ||
Comment 6•6 years ago
|
||
Thanks. Hmm, I don't get this error, I only get the error from bug 1427297.
Pushed by mozilla@jorgk.com: https://hg.mozilla.org/comm-central/rev/a64eb1d3d916 provide better moz-icon for messages attached via drag&drop. r=aceman
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•6 years ago
|
Target Milestone: --- → Thunderbird 59.0
You need to log in
before you can comment on or make changes to this bug.
Description
•