Closed Bug 309406 Opened 19 years ago Closed 19 years ago

Clear Private Data window opens in an odd location on the screen

Categories

(Firefox :: General, defect)

PowerPC
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: jaas, Assigned: jaas)

References

Details

(Keywords: verified1.8)

Attachments

(1 file, 1 obsolete file)

At least with a new profile and no browser windows open, the Clear Private Data
window opens in an odd location on the screen. Even with browser windows open,
it opens in an odd place for me.

Usually the odd place is the very top left or the very bottom right of the
screen. It seems random as to where it shows up. Should probably have the
centerscreen attribute.
Because this is an easy fix and it makes Mac UI look really bad I'm nominating
this for 1.8b5
Flags: blocking1.8b5?
Attached patch fix v1.0 (obsolete) — Splinter Review
Attachment #197118 - Flags: review?(bugs.mano)
Comment on attachment 197118 [details] [diff] [review]
fix v1.0


>+#ifdef XP_MACOSX
>+  // on Mac OS X we don't specify a parent window because it
>+  // makes window positioning weird in some cases
>+  ww.openWindow(null,
>+                "chrome://browser/content/sanitize.xul",
>+                "Sanitize",
>+                "chrome,titlebar,centerscreen,modal",
>+                null);
>+#else
>   ww.openWindow(aParentWindow,
>                 "chrome://browser/content/sanitize.xul",
>                 "Sanitize",
>                 "chrome,titlebar,centerscreen,modal",
>                 null);
>+#endif
> };

You should only #ifdef the first line; plus, the reason we're #ifdefing it is
because we want it to be an app-modal window (the fact that we get a
problematic app modal window instead of a sheet in the case where we do pass a
parent window is a mac widget bug we should file/fix if we're not killing mac
widget rsn).
Attachment #197118 - Flags: review?(bugs.mano) → review-
Attached patch fix v1.1Splinter Review
Attachment #197118 - Attachment is obsolete: true
Attachment #197127 - Flags: review?(bugs.mano)
Comment on attachment 197127 [details] [diff] [review]
fix v1.1

r=mano
Attachment #197127 - Flags: review?(bugs.mano) → review+
Attachment #197127 - Flags: superreview?(mconnor)
Attachment #197127 - Flags: superreview?(mconnor) → superreview+
Flags: blocking1.8b5? → blocking1.8b5+
Attachment #197127 - Flags: approval1.8b5?
Attachment #197127 - Flags: approval1.8b5? → approval1.8b5+
checked in on branch and trunk
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Keywords: fixed1.8
Looks good using Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US;
rv:1.8b5) Gecko/20050927 Firefox/1.4 - the Clear Private Data windows opens in
center. Adding verified keyword.
Keywords: fixed1.8verified1.8
Depends on: 464930
So... this fix leads to some _really_ weird behavior.  See bug 464930.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: