73267 - "Create Profile" dialog displays in wrong position

Status: VERIFIED FIXED

(SeaMonkey :: Startup & Profiles, defect)

Description

[jhp (no longer active)]

The Create Profile dialog displays too far to the right.  While I can 
still see the whole dialog on Win32, on OS/2 it displays partly off 
the edge of the screen.The problem occurs in createProfileWizard.js.  
The Startup() function calls moveToAlertPosition(), which tries to 
center the dialog on the calling window.  However, for some reason, 
the var "outerWidth" of the dialog is set to "1", such that 
moveToAlertPosition tries to center what it thinks is a window of 
width 1.  If I remove these four lines:
  if( window.opener )
    moveToAlertPosition();
  else
    centerWindowOnScreen();
the dialog displays in the 'default' position, which is slightly down 
and indented from the parent window.

I don't know what the goal was with trying to call 
moveToAlertPosition() or centerWindowOnScreen(), but the default 
position is a much better placement for the dialog.

Comment 1

[jhp (no longer active)]

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

Comment 2

[Boris Zbarsky [:bzbarsky]]

On win, mac, linux the call to moveToAlertPosition() correctly centers the
dialog over the calling window...

Looks like an OS/2-only bug

Comment 3

[jhp (no longer active)]

Well, I built a linux build last week, and it also displays the problem:  the
Profile Manager dialog is centered on the screen, but when clicking on "Create
Profile", the left edge of the resulting dialog is in the very middle of Profile
Manager dialog, such that the "Create Profile" dialog goes off the right edge of
the screen.  This is on a resolution of 1024x768.  Attaching screen shot below.

Comment 4

[jhp (no longer active)]

Attachment: Linux screenshot (jpg)

Comment 5

[jhp (no longer active)]

Well, after posting this, i rebuilt on OS/2, and now the "Create 
Profile" dialog comes up correctly centered.  However, i still see the 
problem on linux.  Maybe i just rebuilt wrong.

Marking fixed.

Comment 6

[jhp (no longer active)]

verified as of build 20010424