Tools submenu for seamonkey

pde · pde · commit dee19cee8360 · 2011-07-08T17:29:53.000-07:00
Although this (1) goes in mysterious the Tools-&gt;Web Development menu for some
reason, and this code is deeply inelegant.
diff --git a/src/chrome/content/toolbar_button.js b/src/chrome/content/toolbar_button.js
@@ -41,6 +41,14 @@ function stitch_context_menu() {
     menu.appendChild(popup.cloneNode(true));
   }
 }
+function stitch_context_menu2() {
+  // the same menu appears both under Tools and via the toolbar button:
+  var menu = document.getElementById("https-everywhere-menu2");
+  if (!menu.firstChild) {
+    var popup = document.getElementById("https-everywhere-context");
+    menu.appendChild(popup.cloneNode(true));
+  }
+}
 
 function show_applicable_list(menupopup) {
   var domWin = content.document.defaultView.top;
diff --git a/src/chrome/content/toolbar_button.xul b/src/chrome/content/toolbar_button.xul
@@ -16,6 +16,12 @@
     </menu>
   </menupopup>
 
+  <menupopup id="toolsPopup" onpopupshowing="stitch_context_menu2()">
+    <menu id="https-everywhere-menu2" label="&https-everywhere.about.ext_name;">
+      <!-- "https-everywhere-context" below gets .cloneNode()ed in here -->
+    </menu>
+  </menupopup>
+  
   <toolbarpalette id="BrowserToolbarPalette">
     <toolbarbutton 
       id="https-everywhere-button"

Original file line number	Diff line number	Diff line change
`@@ -41,6 +41,14 @@ function stitch_context_menu() {`
`41`	`41`	`menu.appendChild(popup.cloneNode(true));`
`42`	`42`	`}`
`43`	`43`	`}`
	`44`	`+function stitch_context_menu2() {`
	`45`	`+ // the same menu appears both under Tools and via the toolbar button:`
	`46`	`+ var menu = document.getElementById("https-everywhere-menu2");`
	`47`	`+ if (!menu.firstChild) {`
	`48`	`+ var popup = document.getElementById("https-everywhere-context");`
	`49`	`+ menu.appendChild(popup.cloneNode(true));`
	`50`	`+ }`
	`51`	`+}`
`44`	`52`
`45`	`53`	`function show_applicable_list(menupopup) {`
`46`	`54`	`var domWin = content.document.defaultView.top;`