With linux suite trunk 20050603, if I detach an attachment and then attempt to save it (using save as from the context menu), it "opens" the attachment instead, either in a browser window, or with the how-do-you-want-to-handle-this-file dialog. It should just bring up the file picker and let me save it.
It looks as if we don't really have a way to save detached attachments [Thunderbird also displays the helper dialog for both open and save] unless perhaps we should hand off to saveURL? What would that involve?
saveURL basically only wants the URL (as a string, if I recall correctly). it will then show a filepicker and save the URL.
Comment on attachment 185362 [details] [diff] [review] Possible patch >Index: xpfe/communicator/resources/locale/en-US/contentAreaCommands.properties >=================================================================== >+SaveAttachmentTtle=Save Attachment SaveAttachmentTitle even?
Comment on attachment 185362 [details] [diff] [review] Possible patch this patch works fine for me (with Ttle => Title)
Created attachment 185384 [details] [diff] [review] Proposed patch
Comment on attachment 185384 [details] [diff] [review] Proposed patch I'll take care of this for tbird
Attachment #185384 - Flags: superreview?(bienvenu) → superreview+
Comment on attachment 185384 [details] [diff] [review] Proposed patch I'm not quite happy with SaveAttachmentTitle being in contentAreaCommands.properties, while messenger.properties contains the very same as SaveAttachment, but anyway... While you're at it, you might want to kill the erroneous space before the ellipsis in <!ENTITY detachAttachmentCmd.label "Detach ..."> (http://lxr.mozilla.org/mozilla/search?string=detach%20...) also. ;-) Besides, we should probably have different icons in the attachment box for deleted and detached attachments...
Attachment #185384 - Flags: review?(mnyromyr) → review+
(In reply to comment #8) > Besides, we should probably have different icons in the attachment box for > deleted and detached attachments... Thunderbird already has the former, but missis the latter also: see bug 292385.
Okay, deleted does work in Mozilla, too (my build was horked there).
Attachment #185384 - Flags: approval1.8b3? → approval1.8b3+
Patch checked in.
Status: NEW → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.