Ticket #1383: 1383.patch

_whatsnew.html

@@ -53 +53 @@
                         in the Find/Replace dialog.</li>
                 <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/2124">#2124</a>] PHP file brower: fixed
                         issue with resolving paths on Windows servers with PHP 5.2.4/5.2.5.</li>
+                <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/1383">#1383</a>] Fixed an IE issue where
+                        pressing backspace may merge a hyperlink on the previous line with the text on the current line.
+                        </li>
         </ul>
         <h3>
                 Version 2.6</h3>

editor/_source/classes/fckenterkey.js

@@ -168 +168 @@
                 return false ;
         }
 
+        // Kludge for #1383
+        if ( FCKBrowserInfo.IsIE && isCollapsed )
+        {
+                var endContainer = oRange._Range.endContainer ;
+                var endOffset = oRange._Range.endOffset ;
+                var startNode = oRange.StartNode ;
+
+                var returnNow = (function()
+                {
+                        var brNode = null ;
+                        if ( endContainer.nodeType == 3 )
+                        {
+                                if ( endOffset > 0 )
+                                        return false ;
+
+                                var currentNode = endContainer ;
+                                var commonParent = FCKDomTools.GetCommonParents( endContainer, startNode ).pop() ;
+                                while ( FCKDomTools.CheckIsEditable( currentNode.parentNode )
+                                                && currentNode == currentNode.parentNode.childNodes[0]
+                                                && currentNode.parentNode != commonParent )
+                                        currentNode = currentNode.parentNode ;
+
+                                if ( currentNode.previousSibling && currentNode.previousSibling.nodeName.IEquals( 'br' ) )
+                                        brNode = currentNode.previousSibling ;
+                        }
+                        else if ( endContainer.nodeType == 1 )
+                        {
+                                var currentNode = endContainer.childNodes[ endOffset ] || endContainer.childNodes[ endOffset - 1 ] ;
+                                var commonParent = FCKDomTools.GetCommonParents( currentNode, startNode ).pop() ;
+                                while ( FCKDomTools.CheckIsEditable( currentNode.parentNode )
+                                                && currentNode == currentNode.parentNode.childNodes[0]
+                                                && currentNode.parentNode != commonParent )
+                                        currentNode = currentNode.parentNode ;
+
+                                var prevNode = ( endOffset >= endContainer.length ? currentNode : currentNode.previousSibling ) ;
+
+                                if ( prevNode && prevNode.nodeName.IEquals( 'br' ) )
+                                        brNode = prevNode ;
+                        }
+
+                        if ( brNode )
+                        {
+                                oRange.SetStart( brNode, 3 ) ;
+                                oRange.SetEnd( brNode, 3 ) ;
+                                var bookmark = oRange.CreateBookmark() ;
+                                FCKDomTools.RemoveNode( brNode ) ;
+                                oRange.MoveToBookmark( bookmark ) ;
+                                oRange.Select() ;
+                                return true ;
+                        }
+                })() ;
+
+                if ( returnNow )
+                        return true ;
+        }
+
         var oStartBlock = oRange.StartBlock ;
         var oEndBlock = oRange.EndBlock ;