Closed Bug 269480 Opened 20 years ago Closed 20 years ago

Resizing a window zooms it to the wrong place

Categories

(Core Graveyard :: Widget: Mac, defect)

PowerPC
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: ccarlen, Assigned: ccarlen)

Details

Attachments

(1 file)

In Firefox, choose "Bookmark This Page."
Click the disclosure button to reveal the tree.
Wham! the window moves to the upper-left corner of the screen.

This happens with several windows in Firefox, any where JS is using
window.resizeTo(). The problem is that, on setting the size, XULWindow also sets
the zoom state (?!?!). We often have the wrong rect for the normal user state -
patch coming up.
Attached patch patchSplinter Review
portBounds is where the window used to be - use the rect after it's been moved
and it works.
Attachment #165720 - Flags: review?(pinkerton)
dup of Bug 237136?
Maybe if that bug was assigned to the right component, I would have seen it :-/ It's not a Firefox 
bookmarks bug, but a Mac window bug.

The patch on bug 237136 may fix the visible symptoms, but doesn't fix the underlying problem of 
having the wrong zoom rect. The right fix is probably a combination of the 2 patches.
Comment on attachment 165720 [details] [diff] [review]
patch

r=pink
Attachment #165720 - Flags: review?(pinkerton) → review+
Comment on attachment 165720 [details] [diff] [review]
patch

This will probably fix the buggy zoom command in FF.
Attachment #165720 - Flags: superreview?(sfraser_bugs)
Component: XP Toolkit/Widgets → Widget: Mac
Attachment #165720 - Flags: superreview?(sfraser_bugs) → superreview+
Checked in to trunk.
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: