Bug 1324571 - Use cases of some multi-selection commands in Downloads Library not clear. r=mak
authorPavan <pavankarthikboddeda@gmail.com>
Thu, 19 Jan 2017 01:37:00 +0100
changeset 330278 c8b01174d7139ca83232a099eaf123cdfa0733b7
parent 330277 3d6d5a48deaf7c146b093ae129a0cb75c4334c0c
child 330279 2e438048aa9d6d90b4d0c96a506d7ef01a1053cc
push id85933
push usercbook@mozilla.com
push dateFri, 20 Jan 2017 13:12:19 +0000
treeherdermozilla-inbound@2e438048aa9d [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmak
bugs1324571
milestone53.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1324571 - Use cases of some multi-selection commands in Downloads Library not clear. r=mak
browser/components/downloads/content/allDownloadsViewOverlay.js
--- a/browser/components/downloads/content/allDownloadsViewOverlay.js
+++ b/browser/components/downloads/content/allDownloadsViewOverlay.js
@@ -1156,17 +1156,19 @@ DownloadsPlacesView.prototype = {
     return aCommand == "downloadsCmd_clearDownloads" ||
            document.activeElement == this._richlistbox;
   },
 
   // nsIController
   isCommandEnabled(aCommand) {
     switch (aCommand) {
       case "cmd_copy":
-        return this._richlistbox.selectedItems.length > 0;
+      case "downloadsCmd_openReferrer":
+      case "downloadShowMenuItem":
+        return this._richlistbox.selectedItems.length == 1;
       case "cmd_selectAll":
         return true;
       case "cmd_paste":
         return this._canDownloadClipboardURL();
       case "downloadsCmd_clearDownloads":
         return this._canClearDownloads();
       default:
         return Array.every(this._richlistbox.selectedItems,