Changeset 5511


Ignore:
Timestamp:
05/21/2010 05:56:47 PM (5 years ago)
Author:
garry.yao
Message:

#5719: Avoid firing 'change' event on field when dialog is already closed.

Location:
CKEditor/trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • CKEditor/trunk/CHANGES.html

    r5505 r5511  
    9696                <li><a href="http://dev.fckeditor.net/ticket/5688">#5688</a> : Duplicate ids are used in dialog definition.</li> 
    9797                <li><a href="http://dev.fckeditor.net/ticket/5570">#5570</a> : [IE] First enabling SCAYT blind cursor in editor.</li> 
     98                <li><a href="http://dev.fckeditor.net/ticket/5719">#5719</a> : [IE] 'change' dialog event should not be triggered when dialog is already closed.</li> 
    9899                <li>Updated the following language files:<ul> 
    99100                        <li><a href="http://dev.fckeditor.net/ticket/5432">#5432</a> : Dutch;</li> 
  • CKEditor/trunk/_source/plugins/dialogui/plugin.js

    r5402 r5511  
    8888                                        dialog.on( 'load', function() 
    8989                                                { 
    90                                                         this.getInputElement().on( 'change', function(){ this.fire( 'change', { value : this.getValue() } ); }, this ); 
     90                                                        this.getInputElement().on( 'change', function() 
     91                                                        { 
     92                                                                // Make sure 'onchange' doesn't get fired after dialog closed. (#5719) 
     93                                                                if ( !dialog.parts.dialog.isVisible() ) 
     94                                                                        return; 
     95                                                                 
     96                                                                this.fire( 'change', { value : this.getValue() } ); 
     97                                                        }, this ); 
    9198                                                }, this ); 
    9299                                        this._.domOnChangeRegistered = true; 
Note: See TracChangeset for help on using the changeset viewer.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy