Changeset 8512


Ignore:
Timestamp:
May 24, 2009, 3:48:18 PM (12 years ago)
Author:
charles
Message:

(trunk web) very minor optimizations

Location:
trunk/web/javascript
Files:
2 edited

Legend:

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

    r8505 r8512  
    101101                if ($.browser.safari)
    102102                        this._element.css('margin-top', '7px');
     103
     104                this.initializeTorrentFilesInspectorGroup();
     105
     106                if( data.files ) {
     107                        for( var i=0, row; row=data.files[i]; ++i ) {
     108                                this._file_model[i] = {
     109                                        'index': i,
     110                                        'torrent': this,
     111                                        'length': row.length,
     112                                        'name': row.name
     113                                };
     114                        }
     115                }
     116
     117                // Update all the labels etc
     118                this.refresh(data);
    103119               
    104120                // insert the element
    105121                domParent.appendChild(top_e);
    106 
    107                 this.initializeTorrentFilesInspectorGroup();
    108 
    109                 for( var i=0; data.files!=null && i<data.files.length; ++i ) {
    110                         var src = data.files[i];
    111                         this._file_model[i] = {
    112                                 'index': i,
    113                                 'torrent': this,
    114                                 'length': src.length,
    115                                 'name': src.name
    116                         };
    117                 }
    118 
    119                 // Update all the labels etc
    120                 this.refresh(data);
    121122        },
    122123       
  • trunk/web/javascript/transmission.js

    r8505 r8512  
    7676                }
    7777
    78                 this._torrent_list = $('#torrent_list')[0];
    79                 this._inspector_file_list = $('#inspector_file_list')[0];
    80                 this._inspector_tab_files = $('#inspector_tab_files')[0];
    81                 this._toolbar_buttons = $('#torrent_global_menu ul li');
     78                this._torrent_list             = $('#torrent_list')[0];
     79                this._inspector_file_list      = $('#inspector_file_list')[0];
     80                this._inspector_tab_files      = $('#inspector_tab_files')[0];
     81                this._toolbar_buttons          = $('#torrent_global_menu ul li');
     82                this._toolbar_pause_button     = $('li#pause_selected')[0];
     83                this._toolbar_pause_all_button = $('li#pause_all')[0];
     84                this._toolbar_start_button     = $('li#resume_selected')[0];
     85                this._toolbar_start_all_button = $('li#resume_all')[0];
     86                this._toolbar_remove_button    = $('li#remove')[0];
     87                this._toolbar_delete_button    = $('li#removedata')[0];
     88                this._context_pause_button     = $('li#context_pause_selected')[0];
     89                this._context_start_button     = $('li#context_resume_selected')[0];
    8290               
    8391                // Setup the preference box
     
    288296        {
    289297                var torrents = [ ];
    290                 for( var i=0, len=this._rows.length; i<len; ++i )
    291                         if( this._rows[i]._torrent )
    292                                 if( this._rows[i][0].style.display != 'none' )
    293                                         torrents.push( this._rows[i]._torrent );
     298                for( var i=0, row; row=this._rows[i]; ++i )
     299                        if( row._torrent && ( row[0].style.display != 'none' ) )
     300                                torrents.push( row._torrent );
    294301                return torrents;
    295302        },
     
    299306                var v = this.getVisibleTorrents( );
    300307                var s = [ ];
    301                 for( var i=0, len=v.length; i<len; ++i )
    302                         if( v[i].isSelected( ) )
    303                                 s.push( v[i] );
     308                for( var i=0, row; row=v[i]; ++i )
     309                        if( row.isSelected( ) )
     310                                s.push( row );
    304311                return s;
    305312        },
     
    319326        {
    320327                var rows = [ ];
    321                 for( var i=0, len=this._rows.length; i<len; ++i )
    322                         if( this._rows[i][0].style.display != 'none' )
    323                                 rows.push( this._rows[i] );
     328                for( var i=0, row; row=this._rows[i]; ++i )
     329                        if( row[0].style.display != 'none' )
     330                                rows.push( row );
    324331                return rows;
    325332        },
     
    327334        getTorrentIndex: function( rows, torrent )
    328335        {
    329                 for( var i=0, len=rows.length; i<len; ++i )
    330                         if( rows[i]._torrent == torrent )
     336                for( var i=0, row; row=rows[i]; ++i )
     337                        if( row._torrent == torrent )
    331338                                return i;
    332339                return null;
     
    395402        selectAll: function( doUpdate ) {
    396403                var tr = this;
    397                 for( var i=0, len=tr._rows.length; i<len; ++i )
    398                         tr.selectElement( tr._rows[i] );
     404                for( var i=0, row; row=tr._rows[i]; ++i )
     405                        tr.selectElement( row );
    399406                if( doUpdate )
    400407                        tr.selectionChanged();
     
    402409        deselectAll: function( doUpdate ) {
    403410                var tr = this;
    404                 for( var i=0, len=tr._rows.length; i<len; ++i )
    405                         tr.deselectElement( tr._rows[i] );
     411                for( var i=0, row; row=tr._rows[i]; ++i )
     412                        tr.deselectElement( row );
    406413                tr._last_torrent_clicked = null;
    407414                if( doUpdate )
     
    952959                }
    953960
    954                 for(i = 0; i < torrents.length; ++i ) {
    955                         var t = torrents[i];
     961                for( var i=0, t; t=torrents[i]; ++i ) {
    956962                        sizeWhenDone         += t._sizeWhenDone;
    957963                        sizeDone             += t._sizeWhenDone - t._leftUntilDone;
     
    11951201        {
    11961202                var rows = this.getVisibleRows( );
    1197                 for( var i=0, len=rows.length; i<len; ++i ) {
    1198                         var wasEven = rows[i][0].className.indexOf('even') != -1;
     1203                for( var i=0, row; row=rows[i]; ++i ) {
     1204                        var wasEven = row[0].className.indexOf('even') != -1;
    11991205                        var isEven = ((i+1) % 2 == 0);
    1200                         if( wasEven != isEven ) {
    1201                                 if( wasEven )
    1202                                         rows[i].removeClass('even');
    1203                                 else
    1204                                         rows[i].addClass('even');
    1205                         }
     1206                        if( wasEven != isEven )
     1207                                row.toggleClass('even', isEven);
    12061208                }
    12071209        },
     
    12161218                var upSpeed = 0;
    12171219                var downSpeed = 0;
    1218                 for( var i=0; i<torrentCount; ++i ) {
    1219                         upSpeed += torrents[i].uploadSpeed( );
    1220                         downSpeed += torrents[i].downloadSpeed( );
     1220                for( var i=0, row; row=torrents[i]; ++i ) {
     1221                        upSpeed += row.uploadSpeed( );
     1222                        downSpeed += row.downloadSpeed( );
    12211223                }
    12221224               
     
    14331435        setEnabled: function( key, flag )
    14341436        {
    1435                 if( flag )
    1436                         $(key + '.disabled').removeClass('disabled');
    1437                 else
    1438                         $(key).addClass('disabled');
     1437                $(key).toggleClass( 'disabled', !flag );
    14391438        },
    14401439
     
    14631462                        }
    14641463
    1465                         this.setEnabled( 'li#pause_selected', haveActiveSelection );
    1466                         this.setEnabled( 'li.context_pause_selected', haveActiveSelection );
    1467                         this.setEnabled( 'li#resume_selected', havePausedSelection );
    1468                         this.setEnabled( 'li.context_resume_selected', havePausedSelection );
    1469                         this.setEnabled( 'li#remove', haveSelection );
    1470                         this.setEnabled( 'li#removedata', haveSelection );
    1471                         this.setEnabled( 'li#pause_all', haveActive );
    1472                         this.setEnabled( 'li#resume_all', havePaused );
     1464                        this.setEnabled( this._toolbar_pause_button, haveActiveSelection );
     1465                        this.setEnabled( this._context_pause_button, haveActiveSelection );
     1466                        this.setEnabled( this._toolbar_start_button, havePausedSelection );
     1467                        this.setEnabled( this._context_start_button, havePausedSelection );
     1468                        this.setEnabled( this._toolbar_remove_button, haveSelection );
     1469                        this.setEnabled( this._toolbar_delete_button, haveSelection );
     1470                        this.setEnabled( this._toolbar_pause_all_button, haveActive );
     1471                        this.setEnabled( this._toolbar_start_all_button, havePaused );
    14731472                }
    14741473        }
Note: See TracChangeset for help on using the changeset viewer.