Closed
Bug 204418
Opened 22 years ago
Closed 21 years ago
Firebird menu issues when no open windows exist on OS X
Categories
(Firefox :: Menus, defect)
Tracking
()
RESOLVED
FIXED
Firebird0.8
People
(Reporter: jo.hermans, Assigned: hyatt)
References
Details
Attachments
(1 obsolete file)
When no windows are open, you can see the old Mozilla menubar instead of the
Firebird menubar.
- there are extra Window, Debug and QA menus
- Firebird suddenly knows the cmd-1 shortcut, which doesn't work when a window
is open
- various menus can be seen which are sometimes usable, sometimes not.
- ...
Reporter | ||
Comment 1•22 years ago
|
||
Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.4b) Gecko/20030503
Mozilla Firebird/0.6
*** Bug 206186 has been marked as a duplicate of this bug. ***
*** Bug 206186 has been marked as a duplicate of this bug. ***
Comment 4•21 years ago
|
||
*** Bug 207561 has been marked as a duplicate of this bug. ***
Comment 5•21 years ago
|
||
This problem is due to that "chrome://global/content/hiddenWindow.xul" which
nsAppShellService::CreateHiddenWindow opens is old Mozilla version.
We need new version of hiddenWidnow.xul for Mozilla Firebird. I suggest using
not an overlay as we does today, but XUL preprocessor to create that file from
"browser.xul".
*** Bug 206936 has been marked as a duplicate of this bug. ***
Comment 7•21 years ago
|
||
*** Bug 207664 has been marked as a duplicate of this bug. ***
Comment 8•21 years ago
|
||
kenji, what you say makes it seem to me (as a coder unfamiliar with mozilla
code) that bug 206456 and bug 206936 and bug 206752 all share the same root
cause as this bug. is that right? and if so, someone should do bugzilla magic
to link them all together.
Comment 9•21 years ago
|
||
jonathan:
Hmm, It looks bug 206456 and bug 206752 may have something to do with this, but
bug 206936 seems different bug.
Per bug 206456 and bug 206752, hiddenWindow.xul 's startup routine that is for
SeaMonkey's can breaks correspondence of Mozilla Firebird browser windows'
startup and shutdown routines. Should we make those bugs depends on this, though
I'm not sure that yet? Someone has idea?
Reporter | ||
Comment 10•21 years ago
|
||
The same bug is also present in Thunderbird : bug 214228
Comment 11•21 years ago
|
||
Taking QA Contact as designated owner of Firebird-Menus. Sorry for bugspam.
QA Contact: asa → bugzilla
Comment 13•21 years ago
|
||
*** Bug 215172 has been marked as a duplicate of this bug. ***
Assignee | ||
Updated•21 years ago
|
Status: NEW → ASSIGNED
Target Milestone: --- → Firebird0.7
Assignee | ||
Comment 15•21 years ago
|
||
Mapping out a plan of attack for this bug:
(1) Add a new pref browser.hiddenWindowChromeURL that indicates what the URL of
the hidden window is, so that Firebird and Thunderbird can point to their own
hiddenWindows.
(2) Use the XUL PP to include the menus shared by hiddenWindow into browser.xul for
non-Mac platforms.
(3) Use the XUL PP to place the menus into an overlay for Mac platforms, and then make
the overlay be shared by hiddenWindow.xul and browser.xul in the Mac case only. This
will also eliminate the race condition between hiddenWindow.xul and browser.xul that
leads to an inactive window when you first launch.
Assignee | ||
Comment 16•21 years ago
|
||
Comment 17•21 years ago
|
||
Comment on attachment 129565 [details] [diff] [review]
Implement support for a configurable hiddenWindow.xul file
>+ rv = prefBranch->GetCharPref("browser.hiddenWindowChromeURL", getter_Copies(prefVal));
>+ char* hiddenWindowURL = prefVal.get() ? prefVal.get() : defaultHiddenWindowURL;
hiddenWindowURL should be |const| to match the types of prefVal.get() and
defaultHiddenWindowURL. Also... I can't tell whether prefVal.get() will return
NULL or a pointer to an empty string in the case where the pref doesn't exist.
If you're sure this does the right thing in that case, ok, otherwise you might
change the test to prefVal.IsEmpty().
>+ printf("Hidden window is: %s\n", hiddenWindowURL);
Get rid of the printf before you check in.
r=bryner with those changes.
Attachment #129565 -
Flags: review+
Assignee | ||
Comment 18•21 years ago
|
||
It does do the right thing. I tested both with and without the pref present.
Comment 19•21 years ago
|
||
Comment on attachment 129565 [details] [diff] [review]
Implement support for a configurable hiddenWindow.xul file
a=asa (on behalf of drivers) for checkin to Mozilla 1.5beta.
Assignee | ||
Comment 20•21 years ago
|
||
Fixed, and Firebird now has a hiddenWindow that matches.
Keeping this open so I can do the remaining tasks, which include:
(1) Disabling menu items that are inappropriate when no windows are open.
(2) Repairing some of the commands so that they actually work (e.g., bookmarks).
Assignee | ||
Comment 21•21 years ago
|
||
Comment on attachment 129565 [details] [diff] [review]
Implement support for a configurable hiddenWindow.xul file
Landed this patch on the trunk.
Attachment #129565 -
Attachment is obsolete: true
Assignee | ||
Updated•21 years ago
|
Summary: Mozilla menubar reappears when no open windows → Firebird menu issues when no open windows exist on OS X
Comment 22•21 years ago
|
||
David, might you rethink opening new bugs on the other tasks? I would have liked
to have the "Mozilla menu bar appears" bug FIXED to dupe things against. (no,
not a big deal, just with the total lack of builds it would have been cleaner on
the triage side)
Comment 23•21 years ago
|
||
*** Bug 217339 has been marked as a duplicate of this bug. ***
Updated•21 years ago
|
Target Milestone: Firebird0.7 → Firebird0.8
Comment 24•21 years ago
|
||
Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.6a) Gecko/20031029
Firebird/0.7+; Mac OS 10.2.8
The "File...Open File..." menu dialog does not appear when there are no open
windows.
Comment 25•21 years ago
|
||
"open as tabs" for bookmark folders is missing when no windows are open.
Comment 26•21 years ago
|
||
Whatever the case, the core issue here is fixed. Please file a new bug(s) on
commands that do not work when no window is open.
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Comment 27•21 years ago
|
||
The Mac standard Cmd-~ still cycles through the hidden window,
producing messages suuh as:
###!!! ASSERTION: invalid active window: 'Error', file
../../../../../src/embedding/components/windowwatcher/src/nsWindowWatcher.cpp,
line 886
Break: at file
../../../../../src/embedding/components/windowwatcher/src/nsWindowWatcher.cpp,
line 886
WARNING: getting z level of unregistered window, file
../../../../src/xpfe/appshell/src/nsWindowMediator.cpp, line 636
WARNING: getting z level of unregistered window, file
../../../../src/xpfe/appshell/src/nsWindowMediator.cpp, line 636
Reporter | ||
Comment 28•21 years ago
|
||
Updated•18 years ago
|
QA Contact: bugzilla → menus
You need to log in
before you can comment on or make changes to this bug.
Description
•