Changeset 12790


Ignore:
Timestamp:
Aug 29, 2011, 8:29:45 PM (10 years ago)
Author:
jordan
Message:

#4455 'Action on N>1 torrents doesn't work properly in web interface' -- when right-clicking in the torrent list, don't change the selection if the row we're hovering over is already selected.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/web/javascript/transmission.js

    r12787 r12790  
    290290                        boundingRightPad:  20,
    291291                        boundingBottomPad: 5,
    292                         onContextMenu:     function(e) {
    293                                 var closest_row = $(e.target).closest('.torrent')[0];
    294                                 for (var i=0, row; row = tr._rows[i]; ++i) {
    295                                         if (row.getElement() === closest_row) {
    296                                                 tr.setSelectedRow(row);
    297                                                 break;
    298                                         }
    299                                 }
     292                        onContextMenu: function(ev) {
     293                                var element = $(ev.target).closest('.torrent')[0];
     294                                var i = $('#torrent_list > li').index(element);
     295                                if ((i!==-1) && !tr._rows[i].isSelected())
     296                                        tr.setSelectedRow(tr._rows[i]);
    300297                                return true;
    301298                        }
Note: See TracChangeset for help on using the changeset viewer.