Changeset 624

Timestamp:

2007-08-03 13:25:51 (18 months ago)

Author:

martinkou

Message:

Fixed #1034 : Fixed the issue where hitting Enter in a control selection in IE would send the cursor back to the beginning of the preceding block.

Files:

• FCKeditor/trunk/editor/_source/classes/fckdomrange_ie.js (modified) (1 diff)

FCKeditor/trunk/editor/_source/classes/fckdomrange_ie.js

@@ -108 +108 @@
                         // The following trick is needed so IE makes collapsed selections
                         // inside empty blocks visible (expands the block).
-                        if ( FCKBrowserInfo.IsIE )
+                        var id = ( new Date() ).getTime() + '_select_padding' ;
+                        try
                         {
-                                var id = ( new Date() ).getTime() + '_select_padding' ;
-                                try
-                                {
-                                        oIERange.pasteHTML( '<span id="' + id + '">&nbsp;</span>&nbsp;' ) ;
-                                        oIERange.moveStart( 'character', -1 ) ;
-                                        oIERange.select() ;
-                                        oIERange.pasteHTML( '' ) ;
-                                }
-                                catch (e){}
-                                var span = this.Window.document.getElementById( id ) ;
-                                if ( span )
-                                        span.parentNode.removeChild( span ) ;
+                                oIERange.pasteHTML( '<span id="' + id + '">&nbsp;</span>&nbsp;' ) ;
+                                oIERange.moveStart( 'character', -1 ) ;
                         }
+                        catch (e){}
+                        // The following must be separated into a separate try block
+                        // This fixes #1034.
+                        try
+                        {
+                                oIERange.select() ;
+                                oIERange.pasteHTML( '' ) ;
+                        }
+                        catch (e){}
+                        var span = this.Window.document.getElementById( id ) ;
+                        if ( span )
+                                span.parentNode.removeChild( span ) ;
                 }
                 else