Changeset 1393

Timestamp:

2008-01-23 19:31:52 (2 years ago)

Author:

fredck

Message:

The Focus() call implemented with [1366] is needed for all browsers expect IE. So, I'm not calling it now on IE, which avoids memory leak with IE7.

Files:

• FCKeditor/branches/features/floating_dialog/editor/_source/internals/fckdialog.js (modified) (1 diff)

FCKeditor/branches/features/floating_dialog/editor/_source/internals/fckdialog.js

@@ -178 +178 @@
                         else                                                    // First Dialog.
                         {
-                                FCK.Focus() ;
+                                // Set the Focus in the browser, so the "OnBlur" event is not
+                                // fired. In IE, there is no need to d othat because the dialog
+                                // already moved the selection to the editing area before
+                                // closing (EnsureSelection). Also, the Focus() call here
+                                // causes memory leak on IE7 (weird).
+                                if ( !FCKBrowserInfo.IsIE )
+                                        FCK.Focus() ;
+
                                 this.HideMainCover() ;
                                 topDialog = null ;