Closed Bug 8104 Opened 26 years ago Closed 26 years ago

Profile Manager, application window way too narrow

Categories

(SeaMonkey :: General, defect, P3)

x86
Windows NT

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: cpratt, Assigned: danm.moz)

References

Details

Build ID: 1999061408 Platform: Windows NT At first launch of apprunner, the CPM window is far too narrow; it's only as wide as the word "Communicator".
Summary: Colmmunicator Profile Manager window way too narrow → Profile Manager, application window way too narrow
The same problem exists with the main application window itself.
Assignee: shuang → trudelle
This is a problem with all apprunner windows in today's build; they come up normal size, then resize really narrow. The problem occurs in nsWebShellWindow::SetSizeFromXUL(), where danm is calling sizeString.ToInteger() on an empty string. Becuase of a bug in nsString2 (on which I filed bug 8125), the error code is not being propagated back, and get returned as the result of the function. So we try to set the window width to a large negative number.
Assignee: trudelle → danm
reassigning to danm
rickg is checking in a fix for bug 8125 now, which fill fix the immediate problem. However, you may wish to investigate why nsWebShellWindow::SetSizeFromXUL() is now getting empty strings for height and width, and handle the empty string case more gracefully. Note also the comments in nsString2::ToInteger about passing in the radix.
Mass migration of minor danm bugs from M7 -> M8
Status: NEW → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
Fixed the *real* problem which was in nsString2::ToInteger. I inadvertently set the result value instead of the error code on badly formed input.
*** Bug 8426 has been marked as a duplicate of this bug. ***
Status: RESOLVED → VERIFIED
QA Contact: cpratt
Moving UE/UI component bugs over to new User Interface: Design Feedback component. UE/UI component will be deleted.
Component: UE/UI → User Interface: Design Feedback
Component: User Interface Design → Browser-General
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.