Changeset 14413

Show
Ignore:
Timestamp:
01/14/08 23:47:15 (11 months ago)
Author:
flack
Message:

further performance tuning (only loop through active items in the first place)

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • trunk/midcom/midcom.helper.datamanager2/static/chooser/jquery.chooser_widget.js

    r14412 r14413  
    554554            { 
    555555                // Remove activation from each list element 
    556                 jQuery(this.parentNode).find('li').each(function(i) 
     556                jQuery(this.parentNode).find('li.' + CLASSES.ACTIVE).each(function(i) 
    557557                { 
    558558                    var id = this.id.match(/_([0-9]+)$/); 
    …… 
    661661    function inactivate(id) 
    662662    { 
    663         if (jQuery('#'+options.widget_id + '_result_item_'+id).attr("keep_on_list") == "false" ) 
    664         { 
    665             return; 
    666         } 
    667663        jQuery('#'+options.widget_id + '_result_item_'+id+'_input', list).attr({ value: 0 }); 
    668664        jQuery('#'+options.widget_id + '_result_item_'+id).removeClass(CLASSES.DELETED);