#4455 'Action on N>1 torrents doesn't work properly in web interface' -- when right...
authorJordan Lee <jordan@transmissionbt.com>
Mon, 29 Aug 2011 20:29:45 +0000 (20:29 +0000)
committerJordan Lee <jordan@transmissionbt.com>
Mon, 29 Aug 2011 20:29:45 +0000 (20:29 +0000)
web/javascript/transmission.js

index 061412c0f6fe049fdbfc89316bd6ea4fda2c096c..802de2dbc821f0ce3e7c35ebe2db376abdac92c8 100644 (file)
@@ -289,14 +289,11 @@ Transmission.prototype =
                        boundingElement:   $('div#torrent_container'),
                        boundingRightPad:  20,
                        boundingBottomPad: 5,
-                       onContextMenu:     function(e) {
-                               var closest_row = $(e.target).closest('.torrent')[0];
-                               for (var i=0, row; row = tr._rows[i]; ++i) {
-                                       if (row.getElement() === closest_row) {
-                                               tr.setSelectedRow(row);
-                                               break;
-                                       }
-                               }
+                       onContextMenu: function(ev) {
+                               var element = $(ev.target).closest('.torrent')[0];
+                               var i = $('#torrent_list > li').index(element);
+                               if ((i!==-1) && !tr._rows[i].isSelected())
+                                       tr.setSelectedRow(tr._rows[i]);
                                return true;
                        }
                });