68175 - if rapidly click on Message Filters listbox scroll bar down arrow, Filter Rules dialog pops up

Status: VERIFIED FIXED

(MailNews Core :: Filters, defect)

Description

[ekrock's old account (dead)]

Using N6 Commercial 2001020520 on WinNT 4.0 SP5/6 (not sure).

Using a Messenger that has at least enough filters defined to fill up the Mail
Filters list box and cause the scroll bars to be enabled:

To repro:
1) open Messenger
2) Edit-->Message Filters
3) click as fast as you can about six times on the list box down arrow

Expected: events are cached and handled in order and you scroll down six filters
Actual: Filter Rules dialog pops up, which is undesirable behavior

This is annoying but not fatal. The workaround is to click more slowly on the
scroll arrow.

Comment 1

[Oliver Klee]

The same if you double-click on the little arrows to move the scroll bar.

Comment 2

[Roger Martensson]

Just wanted to say that this annoying bug exists in this release too.

Mozilla 0.9+
Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:0.9+) Gecko/20010515
BuildID: 2001051520

Comment 3

[scottputterman]

reassigning to naving

Comment 4

[laurel]

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

Comment 5

[laurel]

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

Comment 6

[Håkan Waara]

Taking, I've got a fix.

Comment 7

[Håkan Waara]

Attachment: simple fix

Comment 8

[Håkan Waara]

The problem was that the ondoubleclick event fired for the tree where ever one
clicked. So you could double click on a scrollbar and still get the "New Filter"
dialog pop up.

The fix was simply to check that the click came from a treecell. Naving, Seth
please review.

Comment 9

[Navin Gupta]

r=naving for the double-click issue but this does nothing about events being
cached. 

Comment 10

[Håkan Waara]

Sspitzer, can you sr= this little baby?

Comment 11

[(not reading, please use [e-mail] instead)]

sr=sspitzer

after your fix:

1) does double clicking on the column header open up the filter rules dialog?
2) does double clicking on the "enabled cell" (the green dot) open the filter
rules dialog?

if they do, those should be new bugs.

Comment 12

[Håkan Waara]

#1 occurs, but not #2.

Seth, which component should #1 be filed against? XPToolkit?

Comment 13

[(not reading, please use [e-mail] instead)]

no, I think you have to fix #1 is your js.

I think you have to check if the treecell's parent parent is a treerow or a
treeitem (or something like that.)

Comment 14

[Asa Dotzler [:asa]]

a= [e-mail] for checkin to the trunk.
(on behalf of drivers)

Comment 15

[Håkan Waara]

fix checked in with a slightly different fix and sr=sspitzer for that.

Comment 16

[laurel]

OK with june20-21 commercial trunk builds, win98, mac OS 9.0 and linux rh6.2