[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
[or-cvs] [torbutton/master 26/31] Move the context menu.
Author: Mike Perry <mikeperry-git@xxxxxxxxxx>
Date: Mon, 3 Jan 2011 16:53:13 -0600
Subject: Move the context menu.
Commit: 745a57ed69ed4052c89ed6c4d7846142fdd39f1b
FF4 disables the addon-bar, which somehow propagates immutable invisibility
status to the context menu there, so we move it to the nav-bar instead. That
was a fun one to diagnose.
---
src/chrome/content/torbutton.xul | 8 ++++++--
1 files changed, 6 insertions(+), 2 deletions(-)
diff --git a/src/chrome/content/torbutton.xul b/src/chrome/content/torbutton.xul
index 278225c..428d4da 100644
--- a/src/chrome/content/torbutton.xul
+++ b/src/chrome/content/torbutton.xul
@@ -17,6 +17,12 @@
<stringbundle id="torbutton-bundle" src="chrome://torbutton/locale/torbutton.properties"/>
</stringbundleset>
+ <!-- Place the context menu in the nav bar, so that the lack
+ of the status bar on FF4 doesn't make it invisible -->
+ <toolbar id="nav-bar">
+ <popup id="torbutton-context-menu"/>
+ </toolbar>
+
<toolbarpalette id="BrowserToolbarPalette">
<toolbarbutton
id="torbutton-button"
@@ -40,8 +46,6 @@
insertbefore="statusbar-updates"
context="torbutton-context-menu"
onclick="if (event.button == 0) torbutton_toggle(false)"/>
-
- <popup id="torbutton-context-menu"/>
<!-- <statusbarpanel context="clipmenu" class="statusbarpanel-iconic" id="plugins-status" tooltip="plugin-tooltip" onclick="torbutton_toggle_plugins();"/> -->
</statusbar>
<popup id="contentAreaContextMenu">
--
1.7.1