Changeset 877

Show
Ignore:
Timestamp:
2008-03-12 08:26:41 (8 months ago)
Author:
MaierMan
Message:

#608: makeSelection() might be called early

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • branches/1.0.x/chrome/content/dta/select.js

    r832 r877  
    462462        }, 
    463463 
    464         // will be called initially and whenever something changed 
     464 
     465        notify: function() { 
     466                if (this.current) { 
     467                        this.makeSelection(); 
     468                } 
     469        }, 
     470        // will be called initially and whenever something changed       
    465471        makeSelection: function() { 
    466  
    467472                var tree = this.current; 
    468473                var type = tree.type; 

  • branches/1.0.x/chrome/content/dta/select.xul

    r718 r877  
    128128        <vbox style="margin:3px 7px;" class="closer" id="regexcontainer" closed="true" persist="closed"> 
    129129                <caption id="additional" tooltiptext="&additional.tip;" label="&additional.caption;"/> 
    130                 <saveddropdown id="filter" flex="1" tooltiptext="&filter.tip;" preference="filter" allowempty="true" onchange="Dialog.makeSelection();"/> 
    131                 <checkbox id="regex" persist="checked" label="&regex.label;" oncommand="Dialog.makeSelection();" /> 
     130                <saveddropdown id="filter" flex="1" tooltiptext="&filter.tip;" preference="filter" allowempty="true" onchange="Dialog.notify();"/> 
     131                <checkbox id="regex" persist="checked" label="&regex.label;" oncommand="Dialog.notify();" /> 
    132132        </vbox> 
    133133