40522 - remove popupanchor/align params from CreatePopup() on popups to match menus

Status: VERIFIED INVALID

(MailNews Core :: Composition, defect, P3)

Description

[scottputterman]

The autocomplete popup should appear below the addressing widget rather than off 
to the side. 

From 39180, jglick wrote:

"Also, the drop down menu should appear Directly below the addressing field (not 
off to the side like it currently is).  "

Comment 1

[Jean-Francois Ducarroz]

It's a bug of positionment of popup menu. I open the popup menu with -1,-1 as 

position and the edit field as parent. Reassign to pinkerton

Comment 2

[Mike Pinkerton (not reading bugmail)]

i thought i already had this bug, but i can't find it.

Comment 3

[Peter Trudelle]

mass-moving all bugs to m21 that are not +/-

Comment 4

[Mike Pinkerton (not reading bugmail)]

ducarroz, are you saying that if you specify -1, -1 as the position then you 
expect it to go below, but instead it goes to the right? sounds like you are 
expecting one thing when the default is another. hyatt, what was the default, do 
you remember?

is this even a bug?

Comment 5

[Mike Pinkerton (not reading bugmail)]

duzarroz: the way you can get this to work today is to set the |popupanchor| and 
|popupalign| attributes to what you want. Those parameters on |createPopup()| are 
going away to make popups more consistant with menus.

I'll keep this bug around to remove those parameters from createPopup then pass 
this bug over to you to set the attributes.

Comment 6

[Peter Trudelle]

should document the workaround.

Comment 7

[David Hyatt]

This bug needs to be fixed for XUL1.0 (Mozilla 1.0).

Comment 8

[Mike Pinkerton (not reading bugmail)]

riiiiight....which is why i marked it mozilla0.9.

Comment 9

[Mike Pinkerton (not reading bugmail)]

--> gecko

Comment 10

[Mike Pinkerton (not reading bugmail)]

oops!!! wrong bug ;)

Comment 11

[Mike Pinkerton (not reading bugmail)]

since we still need the params on popupBoxObject's CreatePopup() for tooltips and 
context menus, ben and I figured that they should stay on the xbl popup's 
openPopup() method as well. Someone could want to position a popup at a physical 
location and not use the aligment attributes, though we aren't doing that now.

i'm marking this INVALID. Please re-open if anyone disagrees.

Comment 12

[[e-mail] (gone - use [e-mail] instead)]

No objection to Pinkerton's comments were made here, so I'm marking this
VERIFIED INVALID.