From 8d654c3c5f2151a9bae3fc3931036aa030b69297 Mon Sep 17 00:00:00 2001
From: "blakeross%telocity.com"
 <blakeross%telocity.com@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Mon, 30 Sep 2002 04:44:52 +0000
Subject: [PATCH] Revised download UI.

git-svn-id: svn://10.0.0.236/trunk@130783 18797224-902f-48f8-a5cc-f745e15eee43
---
 .../downloads/content/downloadPanel.js        | 17 +++++++++++++
 .../downloads/content/downloadPanel.xul       | 24 +++++++++----------
 .../downloads/locale/downloadmanager.dtd      |  4 ++--
 3 files changed, 31 insertions(+), 14 deletions(-)

diff --git a/mozilla/browser/components/downloads/content/downloadPanel.js b/mozilla/browser/components/downloads/content/downloadPanel.js
index 32bf15d1819..d13bfececd5 100644
--- a/mozilla/browser/components/downloads/content/downloadPanel.js
+++ b/mozilla/browser/components/downloads/content/downloadPanel.js
@@ -219,3 +219,20 @@ function createLocalFile(aFilePath)
   lf.initWithPath(aFilePath);
   return lf;
 }
+
+function buildContextMenu()
+{
+  var selectionCount = gDownloadHistoryView.selectedCount;
+  if (!selectionCount)
+    return false;
+
+  var launchItem = document.getElementById("menuitem_launch");
+  var launchSep = document.getElementById("menuseparator_launch");
+  var removeItem = document.getElementById("menuitem_remove");
+  var showItem = document.getElementById("menuitem_show");
+  showItem.hidden = selectionCount != 1;
+  launchItem.hidden = selectionCount != 1;
+  launchSep.hidden = selectionCount != 1;
+  return true;
+}
+    
diff --git a/mozilla/browser/components/downloads/content/downloadPanel.xul b/mozilla/browser/components/downloads/content/downloadPanel.xul
index 1a1fbb839fd..21b2eb3cdec 100644
--- a/mozilla/browser/components/downloads/content/downloadPanel.xul
+++ b/mozilla/browser/components/downloads/content/downloadPanel.xul
@@ -35,6 +35,16 @@
   <keyset>
     <key id="key_remove" keycode="VK_DELETE" command="cmd_remove"/>
   </keyset>
+  
+  <popup id="downloadHistoryPopup" onpopupshowing="return buildContextMenu();">
+    <menuitem id="menuitem_launch" label="&cmd.openfile.label;" default="true"
+              accesskey="&cmd.openfile.accesskey;" command="cmd_openfile"/>
+    <menuseparator id="menuseparator_launch"/>
+    <menuitem id="menuitem_show" label="&cmd.show.label;" accesskey="&cmd.show.accesskey;"
+              command="cmd_showinshell"/>
+    <menuitem id="menuitem_remove" label="&cmd.remove.label;" accesskey="&cmd.remove.accesskey;"
+              command="cmd_remove"/>
+  </popup>
 
   <vbox flex="1">
     <vbox id="downloadView" flex="3" style="overflow: auto;"
@@ -54,20 +64,10 @@
       <sidebarheader align="center">
         <label value="&finishedDownloads.label;"/>
       </sidebarheader>
-      <toolbar id="download-toolbar" tbalign="stretch" class="chromeclass-toolbar" persist="collapsed">
-        <toolbarbutton label="&cmd.remove.label;" accesskey="&cmd.remove.accesskey;"
-                       command="cmd_remove"/>
-        <toolbarseparator/>
-        <toolbarbutton id="btn_openfile" label="&cmd.openfile.label;" accesskey="&cmd.openfile.accesskey;"
-                       command="cmd_openfile"/>
-        <toolbarbutton id="btn_showinshell" label="&cmd.show.label;"
-                       accesskey="&cmd.show.accesskey;" command="cmd_showinshell"/>
-      </toolbar>
-
       <listbox id="downloadHistoryView" flex="1" class="plain" seltype="multiple"
                 datasources="rdf:null" ref="NC:DownloadsRoot" flags="dont-test-empty"
-                enableColumnDrag="true"
-                onselect="onSelect(event);">
+                enableColumnDrag="true" ondblclick="goDoCommand('cmd_openfile');"
+                onselect="onSelect(event);" context="downloadHistoryPopup">
         <listcols>
           <listcol id="Name" primary="true"
                        label="&name.label;"
diff --git a/mozilla/browser/components/downloads/locale/downloadmanager.dtd b/mozilla/browser/components/downloads/locale/downloadmanager.dtd
index da345587b77..ec84665a3a0 100644
--- a/mozilla/browser/components/downloads/locale/downloadmanager.dtd
+++ b/mozilla/browser/components/downloads/locale/downloadmanager.dtd
@@ -29,8 +29,8 @@
 <!ENTITY cmd.remove.accesskey             "r">
 <!ENTITY cmd.cancel.label                 "Cancel">
 <!ENTITY cmd.cancel.accesskey             "c">
-<!ENTITY cmd.openfile.label               "Launch">
-<!ENTITY cmd.openfile.accesskey           "L">
+<!ENTITY cmd.openfile.label               "Open">
+<!ENTITY cmd.openfile.accesskey           "O">
 <!ENTITY cmd.show.label                   "Show">
 <!ENTITY cmd.show.accesskey               "S">
 <!ENTITY confirmCancel "Are you sure you want to cancel the saving of this file?">