Changeset 7001 for CKEditor/trunk

Timestamp:

06/02/11 14:49:29 (2 years ago)

Author:

garry.yao

Message:

#7915: Make sure IE triggers the "change" event when closing dialog without blurring the input.

Location:

CKEditor/trunk

Files:

• CHANGES.html (modified) (1 diff)
• _source/plugins/dialog/plugin.js (modified) (2 diffs)

CKEditor/trunk/CHANGES.html

@@ -94 +94 @@
                 <li><a href="http://dev.ckeditor.com/ticket/7639">#7639</a> : [IE9] Browser might crash when an object is selected in the document.</li>
                 <li><a href="http://dev.ckeditor.com/ticket/7847">#7847</a> : [IE8] Insert image with non-secure source in a HTTPS page breaks the dialog.</li>
+                <li><a href="http://dev.ckeditor.com/ticket/7915">#7915</a> : [IE] Editing iframe height value missing from the Iframe dialog.</li>
                 <li>Updated the following language files:<ul>
                         <li><a href="http://dev.ckeditor.com/ticket/7834">#7834</a> : Dutch;</li>

CKEditor/trunk/_source/plugins/dialog/plugin.js

@@ -863 +863 @@
                         {
                                 for ( var j in this._.contents[i] )
-                                        fn( this._.contents[i][j] );
+                                        fn.call( this, this._.contents[i][j] );
                         }
                         return this;
@@ -914 +914 @@
                         this.foreach( function( widget )
                                 {
+                                        // Make sure IE triggers "change" event on last focused input before closing the dialog. (#7915)
+                                        if ( CKEDITOR.env.ie && this._.currentFocusIndex == widget.focusIndex )
+                                                widget.getInputElement().$.blur();
+
                                         if ( widget.commit )
                                                 widget.commit.apply( widget, args );