Changeset 12787


Ignore:
Timestamp:
Aug 28, 2011, 7:30:17 PM (10 years ago)
Author:
livings124
Message:

(trunk web) selecting the up arrow when no transfers are selected selects the bottom transfer; eliminate an error when pushing the up/down arrow keys with a blank torrent list

File:
1 edited

Legend:

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

    r12785 r12787  
    482482        {
    483483                var handled = false,
     484                        rows = this._rows,
    484485                    up = ev.keyCode === 38; // up key pressed
    485486                    dn = ev.keyCode === 40, // down key pressed
    486487                    shift = ev.keyCode === 16; // shift key pressed
    487488
    488                 if (up || dn)
     489                if ((up || dn) && rows.length)
    489490                {
    490                         var rows = this._rows,
    491                             last = this.indexOfLastTorrent(),
     491                        var last = this.indexOfLastTorrent(),
    492492                            i = last,
    493493                            anchor = this._shift_index;
    494494
    495                         if (i === -1) // no selection yet
    496                                 i = 0;
    497                         else if (dn)
    498                                 i = (i+1) % rows.length;
    499                         else if (up)
    500                                 i = (i || rows.length) - 1;
     495                        if (dn)
     496                        {
     497                                if (i === -1) // no selection yet
     498                                        i = 0;
     499                                else
     500                                        i = (i+1) % rows.length;
     501                        }
     502                        else
     503                        {
     504                                if (i === -1) // no selection yet
     505                                        i = rows.length - 1;
     506                                else
     507                                        i = (i || rows.length) - 1;
     508                        }
    501509                        var r = rows[i];
    502510
Note: See TracChangeset for help on using the changeset viewer.