From eb3f6131d7de6c413f182b0525be5308e18442b8 Mon Sep 17 00:00:00 2001
From: "mozilla.mano@sent.com" <mozilla.mano@sent.com>
Date: Tue, 21 Aug 2007 22:01:40 -0700
Subject: [PATCH] Bug 345257 - add tooltip to 'all tabs' menu button. patch
 from Michael Ventnor <ventnors_dogs234@yahoo.com.au>, r=me.

---
 browser/base/content/tabbrowser.xml                 | 13 ++++++++++++-
 browser/locales/en-US/chrome/browser/tabbrowser.dtd |  1 +
 .../browser/tabbrowser/tabbrowserBindings.xml       |  3 ++-
 3 files changed, 15 insertions(+), 2 deletions(-)

diff --git a/browser/base/content/tabbrowser.xml b/browser/base/content/tabbrowser.xml
index 48dad66188f6..9555815bca04 100644
--- a/browser/base/content/tabbrowser.xml
+++ b/browser/base/content/tabbrowser.xml
@@ -2452,7 +2452,8 @@
       </xul:arrowscrollbox>
       <xul:hbox class="tabs-alltabs-box" align="center" pack="end" 
                 anonid="alltabs-box">
-        <xul:toolbarbutton class="tabs-alltabs-button" type="menu">
+        <xul:toolbarbutton class="tabs-alltabs-button" type="menu" tooltip="_child">
+          <xul:tooltip anonid="alltabs-tooltip" label="&listAllTabs.label;"/>
           <xul:menupopup class="tabs-alltabs-popup"
                          anonid="alltabs-popup" 
                          position="after_end"/>
@@ -2648,6 +2649,11 @@
         document.getAnonymousElementByAttribute(this, 
                                                 "anonid", "alltabs-box");
       </field>
+      
+      <field name="mAllTabsTooltip">
+        document.getAnonymousElementByAttribute(this, 
+                                                "anonid", "alltabs-tooltip");
+      </field>
 
       <field name="mFlashTimer">null</field>
       <field name="mFlashStage">0</field>
@@ -2705,6 +2711,11 @@
     </implementation>
     <handlers>
       <handler event="TabSelect" action="this._handleTabSelect();"/>
+      <handler event="popupshowing"><![CDATA[
+        if (event.originalTarget == this.mAllTabsTooltip &&
+            document.tooltipNode.parentNode == this.mAllTabsPopup)
+          event.preventDefault();
+      ]]></handler>
     </handlers>
   </binding>
 
diff --git a/browser/locales/en-US/chrome/browser/tabbrowser.dtd b/browser/locales/en-US/chrome/browser/tabbrowser.dtd
index ab09e6d345d6..cd641b8dc2a9 100644
--- a/browser/locales/en-US/chrome/browser/tabbrowser.dtd
+++ b/browser/locales/en-US/chrome/browser/tabbrowser.dtd
@@ -10,3 +10,4 @@
 <!ENTITY  reloadTab.label        "Reload Tab">
 <!ENTITY  reloadTab.accesskey         "r">
 <!ENTITY  newTabButton.tooltip   "Open a new tab">
+<!ENTITY  listAllTabs.label      "List all tabs">
diff --git a/browser/themes/pinstripe/browser/tabbrowser/tabbrowserBindings.xml b/browser/themes/pinstripe/browser/tabbrowser/tabbrowserBindings.xml
index 883d69240267..14d4036e3703 100644
--- a/browser/themes/pinstripe/browser/tabbrowser/tabbrowserBindings.xml
+++ b/browser/themes/pinstripe/browser/tabbrowser/tabbrowserBindings.xml
@@ -64,7 +64,8 @@
             </xul:arrowscrollbox>
             <xul:hbox class="tabs-alltabs-box" pack="end" align="center"
                       anonid="alltabs-box">
-              <xul:toolbarbutton class="tabs-alltabs-button" type="menu">
+              <xul:toolbarbutton class="tabs-alltabs-button" type="menu" tooltip="_child">
+                <xul:tooltip anonid="alltabs-tooltip" label="&listAllTabs.label;"/>
                 <xul:menupopup class="tabs-alltabs-popup"
                   anonid="alltabs-popup" position="after_end"/>
               </xul:toolbarbutton>