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)
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)
Comment 1•24 years ago
|
||
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.
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
Comment 7•24 years ago
|
||
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.
Comment 9•24 years ago
|
||
fixed in the thread pane and folder pane. I'll leave assigned to candice for addressbook.
Comment 10•24 years ago
|
||
verified fixed in threadpane on Mag 062208 mozilla bits.
Assignee | ||
Comment 12•24 years ago
|
||
Fix for address book checked in.
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Comment 13•24 years ago
|
||
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?
Comment 14•24 years ago
|
||
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.
Comment 15•24 years ago
|
||
ok. that makes sense. As always, just trying to push work off to someone else :-)
Comment 16•24 years ago
|
||
*** Bug 43329 has been marked as a duplicate of this bug. ***
Comment 17•24 years ago
|
||
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
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•