338401 - Dragging email over closed folder doesn't open (expand) sub folders

Status: VERIFIED FIXED

(Core :: XUL, defect, P2)

Description

[Ray Booysen]

Windows XP
20060517


I am sure this is a regression.

Before, if you had a folder with subfolders and the tree was closed (I.e. only the parent folder visible) and you dragged an email and held it over the parent folder, the tree of subfolders would open.  As of 20060517, this is not happening anymore.

To reproduce:

1)
Create folder structure:

A
---> B
---> C

2)

Close tree so you can only see:

A

3)

Drag email over A without releasing the mouse.  The tree doesn't open.

What does happen:

You only see A

What should happen:

The tree should open and 

A
--->B
--->C 

should be seen.

Comment 1

[Mike Cowperthwaite]

Reproduced with 3a1-0516, Win2K
Works fine in 2a1-0516.

Comment 2

[Mike Cowperthwaite]

*** Bug 339896 has been marked as a duplicate of this bug. ***

Comment 3

[Mike Cowperthwaite]

I also seen this in Seamonkey 1.5-0518.  As I asked at bug 339474:

Are current trunk Seamonkey builds using Toolkit?  Perhaps this is toolkit
bug 299971?

Comment 4

[Wayne Mery (:wsmwk)]

(In reply to comment #3)
> I also seen this in Seamonkey 1.5-0518.  As I asked at bug 339474:

did these regress at same time?

also, clicking on whitespace of folder line doesn't select a folder, it's now necessary to click on text of folder name (also can't click on message count).  when did the behavior change?

Comment 5

[Mike Cowperthwaite]

(In reply to comment #4)
> (In reply to comment #3)
> also, clicking on whitespace of folder line doesn't select a folder, it's
> now necessary to click on text of folder name (also can't click on message 
> count). when did the behavior change?

Huh.  I filed bug 311533 back in October, about a similar change in behavior 
for the tree's handling of mouse events, on the trunk.  I don't have any builds here that old any longer.

Comment 6

[Mike Cowperthwaite]

(In reply to comment #4)
> also, clicking on whitespace of folder line doesn't select a folder, it's
> now necessary to click on text of folder name (also can't click on message 
> count).  when did the behavior change?

I should have checked this yesterday before posting comment 5, but I don't see that particular symptom with 3a1-0606.

Comment 7

[Wayne Mery (:wsmwk)]

(In reply to comment #6)
> (In reply to comment #4)
> > also, clicking on whitespace of folder line doesn't select a folder, it's
> > now necessary to click on text of folder name (also can't click on message 
> > count).  when did the behavior change?
> 
> I should have checked this yesterday before posting comment 5, but I don't see
> that particular symptom with 3a1-0606.

even weirder, double click DOES work in whitespace (opening a new window BTW). again single click does not 3a1-20060608

Comment 8

[Mike Cowperthwaite]

(In reply to comment #7)
> (In reply to comment #6)
> > (In reply to comment #4)
> > > also, clicking on whitespace of folder line doesn't select a folder, it's
> > > now necessary to click on text of folder name (also can't click on 
> > > message count).  when did the behavior change?
> > 
> > I should have checked this yesterday before posting comment 5, but I don't
> > see that particular symptom with 3a1-0606.
> 
> even weirder, double click DOES work in whitespace (opening a new window BTW).
> again single click does not 3a1-20060608

Wayne, bug 340867 and bug 340811 are about these issue... it's a recent regression, apparently from the patch at bug 296040 (checked in 0606).

Comment 9

[Stephen Donner [:stephend] Not actively reading bugmail]

*** Bug 345582 has been marked as a duplicate of this bug. ***

Comment 10

[Mike Cowperthwaite]

xref bug 343729 (can't scroll a tree by hovering at top or bottom).  That bug is stated to be a regression from bug 326273, which was finally marked Fixed on 2006-05-10, a week before this was opened.

Comment 11

[Wayne Mery (:wsmwk)]

*** Bug 350538 has been marked as a duplicate of this bug. ***

Comment 12

[Mike Cowperthwaite]

*** Bug 357416 has been marked as a duplicate of this bug. ***

Comment 13

[H Beitman]

I had originally filed bug #349344 which was marked as a dup of this one.  I had not been able to get the autoscroll or the auto file open to work since April 25th.

I just did a reinstall of the build of 8/2 and it does autoscroll and the menus open if the cursor is left on them for a few moments.  (I had tried reinstalling it previously without success, so something else might have changed on my computer.)  No builds since 8/2/06 autoscroll on my machine.  Running WinXP SP2.  Hope this helps.

H. Beitman

Comment 15

[Serge Gautherie (:sgautherie)]

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9a8pre) Gecko/200708060303 SeaMonkey/2.0a1pre] (nightly) (W2Ksp4)

(Bug still there, in new SeaMonkey v2.0. (v1.1.4 works fine.))

Comment 16

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Is this a generic tree regression (occurring in other callers that did the same thing), or is it specific to the mail window?

Comment 17

[Serge Gautherie (:sgautherie)]

I can reproduce the same kind of bug within Bookmark Manager, trying to move around a bookmark.

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8.1.9) Gecko/20071030 SeaMonkey/1.1.6] (release) (W2Ksp4)

Folder opens after a short delay.

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9a5pre) Gecko/20070515 SeaMonkey/1.5a] (nightly) (W2Ksp4)
[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9b2pre) Gecko/2007110802 SeaMonkey/2.0a1pre] (nightly) (W2Ksp4)

Folder don't open while hovering.

***

This simply confirms that it is most likely a generic tree regression.
See also bug 337761 comment 16, for example.

(Reminder: root cause of "all" these issues is bug 326273...)

Comment 18

[Neil Deakin]

(In reply to comment #16)
> Is this a generic tree regression (occurring in other callers that did the same
> thing), or is it specific to the mail window?
> 

It's a thread manager regression because timers aren't fired during a drag. Also see bug 389931 where Mats was looking into a problem related to the code that needs changing.  

Comment 19

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Assigning Mats because he's been working on bug 389931 which is very similar to this one.

Comment 20

[Matthias Bockelkamp]

WFM with current tinderbox build

Comment 21

[Serge Gautherie (:sgautherie)]

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9b4pre) Gecko/2008022702 SeaMonkey/2.0a1pre] (nightly) (W2Ksp4)

R.Fixed, by bug 389931.

Comment 22

[Serge Gautherie (:sgautherie)]

V.Fixed, per comment 20 and comment 21.