Changeset 742

Timestamp:

2007-08-29 04:59:49 (15 months ago)

Author:

martinkou

Message:

Fixed #332 : Fixed the issue where setting the caret position to the end of an <a> tag with the mouse doesn't terminate the <a> tag.

Files:

• FCKeditor/trunk/editor/_source/internals/fck_gecko.js (modified) (3 diffs)

FCKeditor/trunk/editor/_source/internals/fck_gecko.js

@@ -69 +69 @@
                         return ;
 
-                var keyCode = evt.keyCode ;
-                // ignore if positioning key is not pressed.
-                // left or up arrow keys need to be processed as well, since <a> links can be expanded in Gecko's editor
-                // when the caret moved left or up from another block element below.
-                if ( keyCode < 33 || keyCode > 40 )
-                        return ;
+                if ( evt.type == 'keypress' )
+                {
+                        var keyCode = evt.keyCode ;
+                        // ignore if positioning key is not pressed.
+                        // left or up arrow keys need to be processed as well, since <a> links can be expanded in Gecko's editor
+                        // when the caret moved left or up from another block element below.
+                        if ( keyCode < 33 || keyCode > 40 )
+                                return ;
+                }
                 
                 var blockEmptyStop = function( node )
@@ -183 +186 @@
                         selection.removeAllRanges() ;
                         selection.addRange( range ) ;
+                        FCK.Events.FireEvent( "OnSelectionChange" ) ;
                 }
                 
@@ -251 +255 @@
         {
                 this.EditorDocument.addEventListener( 'keypress', this._ExecCheckCaret, false ) ;
+                this.EditorDocument.addEventListener( 'click', this._ExecCheckCaret, false ) ;
                 this.AttachToOnSelectionChange( this._ExecCheckEmptyBlock ) ;
         }