Changeset 1079

Timestamp:

2007-11-14 06:32:18 (2 years ago)

Author:

martinkou

Message:

Fixed the issue where the FCKDomRange.CreateBookmark2() algorithm would return wrong bookmark addresses in IE when the caret is placed just before a text node.

Files:

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

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

@@ -461 +461 @@
                         curEnd = curEnd.previousSibling ;
                 }
+
+                // If the object pointed to by the startOffset and endOffset are text nodes, we need
+                // to backtrack and add in the text offset to the bookmark addresses.
+                if ( addrStart.nodeType == 1 && addrStart.childNodes[bookmark.Start[0]].nodeType == 3 )
+                {
+                        var curNode = addrStart.childNodes[bookmark.Start[0]] ;
+                        var offset = 0 ;
+                        while ( curNode.previousSibling && curNode.previousSibling.nodeType == 3 )
+                        {
+                                curNode = curNode.previousSibling ;
+                                offset += curNode.length ;
+                        }
+                        addrStart = curNode ;
+                        bookmark.Start[0] = offset ;
+                }
+                if ( addrEnd.nodeType == 1 && addrEnd.childNodes[bookmark.End[0]].nodeType == 3 )
+                {
+                        var curNode = addrEnd.childNodes[bookmark.End[0]] ;
+                        var offset = 0 ;
+                        while ( curNode.previousSibling && curNode.previousSibling.nodeType == 3 )
+                        {
+                                curNode = curNode.previousSibling ;
+                                offset += curNode.length ;
+                        }
+                        addrEnd = curNode ;
+                        bookmark.End[0] = offset ;
+                }
+
                 // Then, we record down the precise position of the container nodes
                 // by walking up the DOM tree and counting their childNode index