Revert "put all the buttons in a toolbar" 0.8
authorDamyan Ivanov <dmn@debian.org>
Fri, 6 Mar 2009 10:58:50 +0000 (12:58 +0200)
committerDamyan Ivanov <dmn@debian.org>
Fri, 6 Mar 2009 10:58:50 +0000 (12:58 +0200)
This reverts commit 245cbd402b3b0d07ce281851e6305594ece013a9.

Thing is, using a full-width toolbar just to make the buttons
appear after installation is an overkill. The toolbar will be mostly
empty and the user is very likely to move the buttons elsewhere and hide
it.

Instead of this pollution, the extension description shall include sort
instruction where to find the newly available buttons.

Notably, making the buttons appear right after installation is not
feasibly for several reasons:
 * detecting "right after installation" is not trivial
 * finding the right toolbar to add the buttons to is impossible, as
   "right" is user-dependent.

chrome.manifest
chrome/icedeb.xul

index d75a44b..b5e1f7c 100644 (file)
@@ -3,4 +3,3 @@ style chrome://global/content/customizeToolbar.xul chrome://icedeb/content/icede
 
 # Firefox
 overlay chrome://browser/content/browser.xul chrome://icedeb/content/icedeb.xul
-overlay chrome://navigator/content/navigator.xul chrome://icedeb/content/icedeb.xul
index 54c81ac..04bb0dd 100644 (file)
@@ -16,6 +16,7 @@
   <toolbarbutton id="icedeb-deb"/>
   </toolbarpalette>
 
+
 <!-- button details -->
 <toolbarbutton id="icedeb-bts"
   label="Debian BTS"
   class="toolbarbutton-1 chromeclass-toolbar-additional icedeb-deb icedeb-button"
   />
 
-<!-- the toolbar -->
-<toolbox id="navigator-toolbox">
-    <toolbar id="icedeb" class="chromeclass-toolbar"
-        toolbarname="Debian buttons" accesskey="D"
-        mode="icons" iconsize="small"
-        context="toolbar-context-menu"
-        customizable="true" defaultset="icedeb-bts,icedeb-pts,icedeb-deb"
-        >
-    </toolbar>
-</toolbox>
-
 </overlay>