273217 - [Firefox] Lots of menu items are enabled that shouldn't be when no windows are open

Status: VERIFIED FIXED

(Firefox :: Menus, defect)

Description

[Asaf Romano (gone)]

firefox version of bug 75660

Comment 1

[Asaf Romano (gone)]

Attachment: v1

(including the fix for bug 229040 as well)

Comment 2

[Asaf Romano (gone)]

Comment on attachment 171654 [details] [diff] [review]
v1


>+ command="cmd_toogleTaskbar" checked="true" />


<sigh />

Comment 3

[Asaf Romano (gone)]

Attachment: getting better

1) Fix bug 227525 - file -> open file menu item does not work when no windows
open
2) fix typo

Comment 4

[Asaf Romano (gone)]

Attachment: more...

Fix bug 249717 (when no windows open the Go->Home menu does nothing) as well

Comment 5

[Mike Connor [:mconnor]]

Comment on attachment 171826 [details] [diff] [review]
more...

>Index: browser/base/content/browser-menubar.inc
>===================================================================
>RCS file: /cvsroot/mozilla/browser/base/content/browser-menubar.inc,v
>retrieving revision 1.36
>diff -u -p -8 -r1.36 browser-menubar.inc
>--- browser/base/content/browser-menubar.inc	30 Nov 2004 08:22:43 -0000	1.36
>+++ browser/base/content/browser-menubar.inc	20 Jan 2005 01:29:17 -0000

>                 </menu>
>-                <menu label="&charsetMenu.label;" accesskey="&charsetMenu.accesskey;"
>+                <menu id="charsetMenu"
>+                      label="&charsetMenu.label;" accesskey="&charsetMenu.accesskey;"
>                       datasources="rdf:charset-menu" ref="NC:BrowserCharsetMenuRoot"
>                       oncommand="MultiplexHandler(event)" onpopupshowing="CreateMenu('browser');UpdateMenus(event)" onpopupshown="CreateMenu('more-menu');">

lets just fix all of it please.  adding a really short line looks really weird
since we have a really long line, and the giant line really sucks.  Something
like this should be fine.

		<menu id="charsetMenu" label="&charsetMenu.label;"
		      accesskey="&charsetMenu.accesskey;"
oncommand="MultiplexHandler(event)" 
		      datasources="rdf:charset-menu"
ref="NC:BrowserCharsetMenuRoot"
		      onpopupshowing="CreateMenu('browser');UpdateMenus(event)"
		      onpopupshown="CreateMenu('more-menu');">


>+#ifdef XP_MACOSX
>+// The following functions should be used by both hiddenWindow.xul and windows which
>+// don't have menus on platforms other than Mac OS X

This comment didn't make sense the first time I read it.  Double negatives are
bad.  Try the following

// The following functions should be used by both hiddenWindow.xul 
// and any other windows which only have menus on Mac OS X


Plus the viewSidebarMenu id issue from IRC (breaking all sidebar extensions is
a bad thing!) and we should be done here.

Comment 6

[Asaf Romano (gone)]

Attachment: patch

Comment 7

[Mike Connor [:mconnor]]

Comment on attachment 172017 [details] [diff] [review]
patch

>@@ -1223,16 +1273,23 @@
>   case "window":
>     OpenBrowserWindow();
>     break;
>   }
> }
> 
> function loadOneOrMoreURIs(aURIString)
> {
>+#ifdef XP_MACOSX
>+  if (window.location.href != getBrowserURL())
>+  {
>+    newWindow = openDialog(getBrowserURL(), "_blank", "all,dialog=no", aURIString);
>+    return;
>+  }
>+#endif
>   var urls = aURIString.split("|");


r=me with a comment here explaining what we're doing.  (i.e. "we're not a
browser window, pass the URI string to a new browser window")

Comment 8

[Asaf Romano (gone)]

checked in

Comment 9

[Frankie]

Domo Arigato, Asaf Romano, for doing the job nobody wanted to. Thank you thank you.

I'd love to verify these as soon as a patched nightly comes up (latest is Jan 20).

Comment 10

[Frankie]

v Firefox 20050128.

Comment 11

[José Jeria]

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

Comment 12

[Jason Tessier]

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

Comment 13

[Jason Tessier]

What version is this fix in? Bug 292018 was marked as a duplicate of this and
still happens (Firefox 1.06 OS X). Also another similar issue occurs by doing this:

1. Launch Firefox.
2. Open a new browser window. Download something big (optional)
3. Open the downloads window.
4. Close the original browser window.
5. Try to open another browser window while the download window is open.

Mac OS X 10.3.9

Comment 14

[Jaime Mitchell (use [e-mail] for email)]

(In reply to comment #13)
> What version is this fix in? 

It will be fixed in Firefox 1.5. Security releases (1.0.6 etc..) only get
security fixes.

Comment 15

[Asaf Romano (gone)]

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