(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.
This commit is contained in:
parent
998c6e6e8e
commit
ae8335bf7a
|
@ -78,6 +78,7 @@ Transmission.prototype =
|
||||||
this._torrent_list = $('#torrent_list')[0];
|
this._torrent_list = $('#torrent_list')[0];
|
||||||
this._inspector_file_list = $('#inspector_file_list')[0];
|
this._inspector_file_list = $('#inspector_file_list')[0];
|
||||||
this._inspector_tab_files = $('#inspector_tab_files')[0];
|
this._inspector_tab_files = $('#inspector_tab_files')[0];
|
||||||
|
this._toolbar_buttons = $('#torrent_global_menu ul li');
|
||||||
|
|
||||||
// Setup the preference box
|
// Setup the preference box
|
||||||
this.setupPrefConstraints();
|
this.setupPrefConstraints();
|
||||||
|
@ -1443,11 +1444,9 @@ Transmission.prototype =
|
||||||
updateButtonStates: function()
|
updateButtonStates: function()
|
||||||
{
|
{
|
||||||
var showing_dialog = new RegExp("(prefs_showing|dialog_showing|open_showing)").test(document.body.className);
|
var showing_dialog = new RegExp("(prefs_showing|dialog_showing|open_showing)").test(document.body.className);
|
||||||
if (showing_dialog)
|
this._toolbar_buttons.toggleClass( 'disabled', showing_dialog );
|
||||||
{
|
|
||||||
$('.torrent_global_menu ul li').addClass('disabled');
|
if (!showing_dialog)
|
||||||
}
|
|
||||||
else
|
|
||||||
{
|
{
|
||||||
var torrents = this.getVisibleTorrents( );
|
var torrents = this.getVisibleTorrents( );
|
||||||
var haveSelection = false;
|
var haveSelection = false;
|
||||||
|
@ -1466,8 +1465,6 @@ Transmission.prototype =
|
||||||
if( isSelected && !isActive ) havePausedSelection = true;
|
if( isSelected && !isActive ) havePausedSelection = true;
|
||||||
}
|
}
|
||||||
|
|
||||||
$('.torrent_global_menu ul li.disabled').removeClass('disabled');
|
|
||||||
|
|
||||||
this.setEnabled( 'li#pause_selected', haveActiveSelection );
|
this.setEnabled( 'li#pause_selected', haveActiveSelection );
|
||||||
this.setEnabled( 'li.context_pause_selected', haveActiveSelection );
|
this.setEnabled( 'li.context_pause_selected', haveActiveSelection );
|
||||||
this.setEnabled( 'li#resume_selected', havePausedSelection );
|
this.setEnabled( 'li#resume_selected', havePausedSelection );
|
||||||
|
|
Loading…
Reference in New Issue