278377 - Add "Window" and "Help" menus to "View Source" and "DOM Inspector" windows

Status: RESOLVED FIXED

(Firefox :: Menus, defect, P2)

Description

[Hiro]

There is no item of Window and Help in the menu when "View Page Source" and
"DOM Inspactor" are executed. 
I think that there should be a standard menu item. 

Reproducible: Always

Mac OS X 10.3.7
Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; ja-JP; rv:1.8a6) Gecko/20050112
Firefox/1.0+

Comment 1

[Hiro]

Attachment: screen shot of View Page Source menu

Comment 2

[Hiro]

Attachment: screen shot of DOM Inspactor menu

Comment 3

[O. Atsushi (Torisugari)]

*** This bug has been marked as a duplicate of 267227 ***

Comment 4

[Asaf Romano (gone)]

not a dupe, only dependent

Comment 5

[Asaf Romano (gone)]

Attachment: Fix DOMi

Also, i just realized: the "Promote Firefox" menuitem doesn't work in
non-browser windows, I fixed that as well (by moving getUILink to utility
overlay and moving the reginon string bundle to baseMenuOverlay).

Comment 6

[Mike Connor [:mconnor]]

Comment on attachment 178680 [details] [diff] [review]
Fix DOMi

>===================================================================
>RCS file: /cvsroot/mozilla/browser/extensions/inspector/content/inspector.xul,v
>retrieving revision 1.2
>diff -u -p -8 -r1.2 inspector.xul
>--- browser/extensions/inspector/content/inspector.xul	22 May 2004 20:59:00 -0000	1.2
>+++ browser/extensions/inspector/content/inspector.xul	26 Mar 2005 19:45:19 -0000

>@@ -69,16 +71,19 @@
>     <command id="cmd_quit"/>
>   </commandset>
> 
>   <keyset id="tasksKeys">
>     <key id="key_closeInspector" key="&closeCmd.key;" command="cmdClose"
>          modifiers="accel"/>
>   </keyset>
> 
>+  <commandset id="baseMenuCommandSet"/>
>+  <keyset id="baseMenuKeyset"/>
>+

nit: move the commandset to below the other commandset.

r=me

Comment 7

[Asaf Romano (gone)]

leavin open for View Source.

Checking in base/content/baseMenuOverlay.xul;
/cvsroot/mozilla/browser/base/content/baseMenuOverlay.xul,v  <-- 
baseMenuOverlay.xul
new revision: 1.2; previous revision: 1.1
done
Checking in base/content/browser-sets.inc;
/cvsroot/mozilla/browser/base/content/browser-sets.inc,v  <--  browser-sets.inc
new revision: 1.43; previous revision: 1.42
done
Checking in base/content/browser.js;
/cvsroot/mozilla/browser/base/content/browser.js,v  <--  browser.js
new revision: 1.405; previous revision: 1.404
done
Checking in base/content/utilityOverlay.js;
/cvsroot/mozilla/browser/base/content/utilityOverlay.js,v  <--  utilityOverlay.js
new revision: 1.18; previous revision: 1.17
done
Checking in components/bookmarks/content/bookmarksManager.xul;
/cvsroot/mozilla/browser/components/bookmarks/content/bookmarksManager.xul,v 
<--  bookmarksManager.xul
new revision: 1.27; previous revision: 1.26
done
Checking in extensions/inspector/jar.mn;
/cvsroot/mozilla/browser/extensions/inspector/jar.mn,v  <--  jar.mn
new revision: 1.7; previous revision: 1.6
done
Checking in extensions/inspector/content/inspector.xul;
/cvsroot/mozilla/browser/extensions/inspector/content/inspector.xul,v  <-- 
inspector.xul
new revision: 1.3; previous revision: 1.2
done
Checking in extensions/inspector/content/toolboxOverlay.xul;
/cvsroot/mozilla/browser/extensions/inspector/content/toolboxOverlay.xul,v  <--
 toolboxOverlay.xul
new revision: 1.3; previous revision: 1.2
done

Comment 8

[Asaf Romano (gone)]

Attachment: view source / view selection source fix

Comment 9

[Asaf Romano (gone)]

Attachment: Fix "View [Partial] Source" windows

Comment 10

[Asaf Romano (gone)]

Checking in browser/base/jar.mn;
/cvsroot/mozilla/browser/base/jar.mn,v  <--  jar.mn
new revision: 1.91; previous revision: 1.90
done
Checking in toolkit/components/viewsource/content/viewPartialSource.xul;
/cvsroot/mozilla/toolkit/components/viewsource/content/viewPartialSource.xul,v 
<--  viewPartialSource.xul
new revision: 1.20; previous revision: 1.19
done
Checking in toolkit/components/viewsource/content/viewSource.xul;
/cvsroot/mozilla/toolkit/components/viewsource/content/viewSource.xul,v  <-- 
viewSource.xul
new revision: 1.24; previous revision: 1.23
done
Checking in viewSourceOverlay.xul;
/cvsroot/mozilla/browser/base/content/viewSourceOverlay.xul,v  <-- 
viewSourceOverlay.xul
new revision: 1.6; previous revision: 1.5
done