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