Closed Bug 43134 Opened 24 years ago Closed 24 years ago

Dragging scrollbar thumb in the threadPane initiates a drag and drop session

Categories

(SeaMonkey :: MailNews: Message Display, defect, P3)

x86
Windows 95
defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: jrgmorrison, Assigned: chuang)

References

Details

(Whiteboard: [nsbeta2+])

Dragging scrollbar thumb in the threadPane initiates a drag and drop session

This is with the new tree code fully turned on. On one occasion, this crashed
on me. (By the way, this bug has behaviour identical to bug #41186, although
that was for the regular scrollbars in the browser). 

Steps to Reproduce:

1) start mozilla -mail, and bring up the contents of a folder in the
   thread pane

2) grab the scrollbar thumb and drag it down

Actual Results:
  You get the icon for 'drag in progress' showing, and this exception
  comes out in the console.

JavaScript error:
 line 0: uncaught exception: [Exception... "Component returned failure code: 
0x80004005 (NS_ERROR_FAILURE) [nsIDragService.invokeDragSession]"  nsresult: 
"0x80004005 (NS_ERROR_FAILURE)"  location: "JS frame :: 
chrome://messenger/content/messengerdnd.js :: BeginDragThreadTree :: line 144" 
 data: no]

Build Date & Platform Bug Found:
  2000061920 win95 comm. verif build (after the tree landing)
 
Additional Builds and Platforms Tested On:
  I don't see this in my own linux tree built about the same time;
  (not sure why not)

Additional Information:

(perhaps the early bailout test in function BeginDragThreadTree of
messengerdnd.js is no longer making the right comparison now that
<tree> moved to XBL)
Just to head this off, THIS IS NOT MY BUG.  The new tree widget properly allows 
the scrollbar events to go through the DOM (the old tree widget improperly ate 
them), and your event handler should check event.target to make sure it's 
actually a treecell.
*** Bug 43158 has been marked as a duplicate of this bug. ***
*** Bug 43184 has been marked as a duplicate of this bug. ***
Scott had reassigned one of the dup bugs to candice to look at.  I'll do so 
here.
Assignee: putterman → chuang
nominate for nsbeta2.  Comments in the duplicate bugs indicate that having this 
"drag and drop" action gets in the way of actually scrolling in the thread pane.
This is a highly visible area of the mail product and something easily 
encountered (if the user has a number of messages which exceed the height of the 
thread pane).
Keywords: nsbeta2
*** Bug 43185 has been marked as a duplicate of this bug. ***
this is XP.  I see it on Mac OS9 6/21 moz bits.
Alec has teh fix and I need to do the same thing for address book results pane 
too.  We need to add the checking for "treecell" and "treeitem" in drag & drop 
code with the new tree widget.
fixed in the thread pane and folder pane. I'll leave assigned to candice for
addressbook.
verified fixed in threadpane on Mag 062208 mozilla bits.
Putting on [nsbeta2+] radar for beta2 fix. 
Whiteboard: [nsbeta2+]
Fix for address book checked in.
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Not that this matters anymore since it was an easy fix, but if every tree that 
wants to do drag and drop and have a scrollbar is going to have this problem and 
given that it's highly unlikely that anyone actually wants a tree with a 
scrollbar that can be dragged anywhere outside of the scrollbar area, is there 
some way we could make it so that every tree implementor doesn't have to do 
this?
Well, you really should be checking the event target, since - even 
ignoring the scrollbar - someone could always click either in your tree header 
or in the blank space of a tree.
ok.  that makes sense.  As always, just trying to push work off to someone else 
:-)
*** Bug 43329 has been marked as a duplicate of this bug. ***
OK using 2000-06-26-08 m17 commercial build, linux rh6.0, NT 4.0, mac OS 9.0
Status: RESOLVED → VERIFIED
QA Contact: lchiang → laurel
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.