Can't drag messages to attachment pane

VERIFIED FIXED

Status

MailNews Core
Composition
--
major
VERIFIED FIXED
17 years ago
10 years ago

People

(Reporter: shaver, Assigned: (not reading, please use seth@sspitzer.org instead))

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

I dunno about other platforms, but on Linux I can't drag a message from the
thread pane into the little white Attachment box.  When I drop it, I get the
``rejection slide'' back to the thread pane.
reassign to varada
Assignee: ducarroz → varada

Comment 2

17 years ago
Ditto with Windows ME

Comment 3

17 years ago
WFM using Mac/2001091311 (0.9.4).

Comment 4

16 years ago
*** Bug 102590 has been marked as a duplicate of this bug. ***

Comment 5

16 years ago
I think the last WFM comments are misplaced... doesn't work on mac (os X or 9.1)
on the branch.  
OS: Linux → All
Hardware: PC → All
taking.  fix in hand (as a favor for shaver).
Assignee: varada → sspitzer
Created attachment 51621 [details] [diff] [review]
the fix, at least for now.
the problem is that a while ago we switched from having different flavors (one 
for messages, one for folders) to one generic type.

this fix gets message dnd to attachment pane working again, but it also allows 
you to drop folders.  if you did such a thing, we alert on sending letting you 
know that we couldn't attach the folder (we give the whole folder uri).

one day, when we clean up messengerdnd.js to have multiple js flavors, we'll 
fix this code again to just work for messeages.  (see #72617 for the cleanup.)  
I'll log a new bug (depending on #72617) so we don't forget.

can I get some reviews?
Status: NEW → ASSIGNED

Comment 9

16 years ago
r=naving
fixed on the trunk.

see #102630 for the new bug (about folders)
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
Reopening.  This doesn't work on Linux.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Created attachment 57425 [details] [diff] [review]
fix case in flavour

This fixes the case on the flavour and gets message dragging working on Linux.

Comment 13

16 years ago
Comment on attachment 57425 [details] [diff] [review]
fix case in flavour

r=jag (ew, shouldn't that flavour be prepended with x-moz-, and possible better
formatted? /me goes to file a bug)
Attachment #57425 - Flags: review+

Comment 14

16 years ago
Comment on attachment 57425 [details] [diff] [review]
fix case in flavour

r=rginda
Blocks: 104864
Comment on attachment 57425 [details] [diff] [review]
fix case in flavour

sr=sspitzer

thanks blizzard.  

I guess windows code is case insensitive. 

jag logged a bug on me to fix the flavor (x-moz-*) and I'll worry about that
next.


I'm in the middle of fixing dnd for the addressbook re-write, so I'll keep both
issues in mind.
Attachment #57425 - Flags: superreview+
Comment on attachment 57425 [details] [diff] [review]
fix case in flavour

a=dbaron for 0.9.6
Attachment #57425 - Flags: approval+
Fix checked into the trunk and the 0.9.6 branch.
Status: REOPENED → RESOLVED
Last Resolved: 16 years ago16 years ago
Resolution: --- → FIXED

Updated

16 years ago
QA Contact: esther → trix

Comment 18

16 years ago
verified fix w/ 2002022203 on linux, mac, & WIN32
Status: RESOLVED → VERIFIED
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.