]> git.ktnx.net Git - icedeb.git/commitdiff
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 d75a44b684e6046b2a9a8bc0347b34b191e7e666..b5e1f7cd5489edc0c2b12ccb8adf54ab5f272bf8 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
 
 # Firefox
 overlay chrome://browser/content/browser.xul chrome://icedeb/content/icedeb.xul
-overlay chrome://navigator/content/navigator.xul chrome://icedeb/content/icedeb.xul
index 54c81ace161472260ec77e2b244df8e04d77f42f..04bb0dd8bbc697bfab2898fb58356b90284a585d 100644 (file)
@@ -16,6 +16,7 @@
   <toolbarbutton id="icedeb-deb"/>
   </toolbarpalette>
 
   <toolbarbutton id="icedeb-deb"/>
   </toolbarpalette>
 
+
 <!-- button details -->
 <toolbarbutton id="icedeb-bts"
   label="Debian BTS"
 <!-- button details -->
 <toolbarbutton id="icedeb-bts"
   label="Debian BTS"
   class="toolbarbutton-1 chromeclass-toolbar-additional icedeb-deb icedeb-button"
   />
 
   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>
 
 </overlay>