Changeset 8502


Ignore:
Timestamp:
May 23, 2009, 9:39:16 PM (12 years ago)
Author:
charles
Message:

(trunk web) building an array of toolbar buttons is sort of expensive in jQuery, so just do it once on startup, and keep that copy.

File:
1 edited

Legend:

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

    r8500 r8502  
    7979                this._inspector_file_list = $('#inspector_file_list')[0];
    8080                this._inspector_tab_files = $('#inspector_tab_files')[0];
     81                this._toolbar_buttons = $('#torrent_global_menu ul li');
    8182               
    8283                // Setup the preference box
     
    14441445        {
    14451446                var showing_dialog = new RegExp("(prefs_showing|dialog_showing|open_showing)").test(document.body.className);
    1446                 if (showing_dialog)
    1447                 {
    1448                         $('.torrent_global_menu ul li').addClass('disabled');
    1449                 }
    1450                 else
     1447                this._toolbar_buttons.toggleClass( 'disabled', showing_dialog );
     1448
     1449                if (!showing_dialog)
    14511450                {
    14521451                        var torrents = this.getVisibleTorrents( );
     
    14671466                        }
    14681467
    1469                         $('.torrent_global_menu ul li.disabled').removeClass('disabled');
    1470 
    14711468                        this.setEnabled( 'li#pause_selected', haveActiveSelection );
    14721469                        this.setEnabled( 'li.context_pause_selected', haveActiveSelection );
Note: See TracChangeset for help on using the changeset viewer.