Changeset 8502 for trunk/web/javascript

Timestamp:

May 23, 2009, 9:39:16 PM (13 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:

• trunk/web/javascript/transmission.js (modified) (3 diffs)

trunk/web/javascript/transmission.js

@@ -79 +79 @@
                 this._inspector_file_list = $('#inspector_file_list')[0];
                 this._inspector_tab_files = $('#inspector_tab_files')[0];
+                this._toolbar_buttons = $('#torrent_global_menu ul li');
                 
                 // Setup the preference box
@@ -1444 +1445 @@
         {
                 var showing_dialog = new RegExp("(prefs_showing|dialog_showing|open_showing)").test(document.body.className);
-                if (showing_dialog)
-                {
-                        $('.torrent_global_menu ul li').addClass('disabled');
-                }
-                else
+                this._toolbar_buttons.toggleClass( 'disabled', showing_dialog );
+
+                if (!showing_dialog)
                 {
                         var torrents = this.getVisibleTorrents( );
@@ -1467 +1466 @@
                         }
 
-                        $('.torrent_global_menu ul li.disabled').removeClass('disabled');
-
                         this.setEnabled( 'li#pause_selected', haveActiveSelection );
                         this.setEnabled( 'li.context_pause_selected', haveActiveSelection );