Changeset 878

Show
Ignore:
Timestamp:
2008-03-12 08:30:18 (10 months ago)
Author:
MaierMan
Message:

#608: makeSelection() might be called early

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • trunk/chrome/content/dta/select.js

    r833 r878  
    465465        }, 
    466466 
    467         // will be called initially and whenever something changed 
     467 
     468        notify: function() { 
     469                if (this.current) { 
     470                        this.makeSelection(); 
     471                } 
     472        }, 
     473        // will be called initially and whenever something changed       
    468474        makeSelection: function() { 
    469  
    470475                var tree = this.current; 
    471476                var type = tree.type; 

  • trunk/chrome/content/dta/select.xul

    r796 r878  
    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();"/> 
     130                <saveddropdown id="filter" flex="1" tooltiptext="&filter.tip;" preference="filter" allowempty="true" onchange="Dialog.notify();"/> 
    131131        </vbox> 
    132132