154880 - drop down lists do not go away when the mousedown occurs outside of it

Status: VERIFIED FIXED

(Core Graveyard :: Embedding: Mac, defect)

Description

[Conrad Carlen (not reading bugmail)]

Happens with PPEmbed on any dropdown in bugzilla.

Comment 1

[amutch]

Is this a case of the menu never going away or only when the next
mousedown-click occurs? If it is the second case, you can see the same behavior
in Mozilla 1.0 and the currently nightly build of MFCEmbed. I did notice in IE6
that the menu goes away as soon as you mouse-up outside the menu. 

Comment 2

[Conrad Carlen (not reading bugmail)]

Attachment: patch

This fixes it. A mousedown anywhere other than in the pop-up list itself causes
the pop-up window to be rolled up.

Comment 3

[Conrad Carlen (not reading bugmail)]

CC'ing Simon for sr=.

Also, making summary more accurate. It's not the mouseup that should cause one
of these windows to be rolled up - only the next mousedown.

Comment 4

[Mike Pinkerton (not reading bugmail)]

Comment on attachment 89783 [details] [diff] [review]
patch

r=pink

Comment 5

[Simon Fraser [no longer active]]

Comment on attachment 89783 [details] [diff] [review]
patch

sr=sfraser

Comment 6

[Conrad Carlen (not reading bugmail)]

Fixed. I need verification on this and to get it into 1.0.1.

Comment 7

[Jaime Rodriguez, Jr.]

Does this need to be on the 1.0 branch for the 1.0.2 milestone, or is this
already on the 1.0 branch?

Comment 8

[Conrad Carlen (not reading bugmail)]

Eek! It's not fixed on the branch. Just tried my branch build from today and it
exhibits this problem. Verification on the trunk never came.

Comment 9

[Jaime Rodriguez, Jr.]

My assumption is that this is needed for a major mac embedding customer, since
it is a dependency of bug 150046. Hence, I am marking this bug as topembed+,
nsbeta1+, and nominating the patch for the 1.0.2 milestone.

Comment 10

[Randell Jesup [:jesup] (needinfo me)]

Comment on attachment 89783 [details] [diff] [review]
patch

a=[e-mail] for 1.0 branch.  Change mozilla1.0.2+ to fixed1.0.2 when
checked in

Comment 11

[Jaime Rodriguez, Jr.]

edt1.0.2+ (on EDT's behalf) for checkin to the 1.0 branch. pls check this in
asap, then replace "mozilla1.0.2+" with "fixed1.0.2". thanks!

Comment 12

[Conrad Carlen (not reading bugmail)]

Checked into branch.

Comment 13

[Jaime Rodriguez, Jr.]

mdunn: can you pls verify this as fixed on the 1.0 branch, then replace
"fixed1.0.2" with "verified1.0.2". thanks!

Comment 14

[Michael Dunn]

Mazie, Sheela
please verify this on the branch and change keyword to verified1.0.2

Comment 15

[Gayatri Rimola]

Please verify the bug. Once verified, change the keyword fixed1.0.2 to
verified1.0.2 

Comment 16

[Michael Dunn]

Sheela can you try to verfy this bug on the branch... Mazie is having access
problems.

Comment 17

[Sheela Ravindran]

Checked to see the bug happens on the older builds from 2002-09-16

Verifying that this is now fixed on 2002-09-25 PPEmbed build on mac os x
Changing the keyword from fixed1.0.2

Not changing the bug status to verified since this needs to be checked on trunk.

Comment 18

[Sheela Ravindran]

Marking verified on build: mozilla -2002-10-02 on PPEmbed.

The drop down windows now roll up with the next mouse down..
Tested on bugzilla query page.