Changeset 1996

Timestamp:

2008-05-14 13:57:13 (8 months ago)

Author:

fredck

Message:

Fixed #1383 : Fixed an IE issue where pressing backspace may merge a hyperlink on the previous line with the text on the current line.

Location:

FCKeditor/trunk

Files:

• editor/_source/classes/fckenterkey.js (modified) (1 diff)
• _whatsnew.html (modified) (1 diff)

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

@@ -169 +169 @@
         }
 
+        // On IE, it is better for us handle the deletion if the caret is preceeded
+        // by a <br> (#1383).
+        if ( FCKBrowserInfo.IsIE )
+        {
+                var previousElement = FCKDomTools.GetPreviousSourceElement( oRange.StartNode, true ) ;
+
+                if ( previousElement && previousElement.nodeName.toLowerCase() == 'br' )
+                {
+                        // Create a range that starts after the <br> and ends at the
+                        // current range position.
+                        var testRange = oRange.Clone() ;
+                        testRange.SetStart( previousElement, 4 ) ;
+
+                        // If that range is empty, we can proceed cleaning that <br> manually.
+                        if ( testRange.CheckIsEmpty() )
+                        {
+                                previousElement.parentNode.removeChild( previousElement ) ;
+                                return true ;
+                        }
+                }
+        }
+
         var oStartBlock = oRange.StartBlock ;
         var oEndBlock = oRange.EndBlock ;

FCKeditor/trunk/_whatsnew.html

@@ -89 +89 @@
                 <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/2184">#2184</a>] Fixed several validation 
                         errors in the File Browser.</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>