Changeset 139

Timestamp:

2007-02-17 13:01:31 (20 months ago)

Author:

fredck

Message:

Fixed #81. IE has (another) DOM bug, which caused FCKDomTools.RTrimNode to break.

Files:

• FCKeditor/trunk/editor/_source/internals/fckdomtools.js (modified) (1 diff)

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

@@ -89 +89 @@
                                         if ( sTrimmed.length == 0 )
                                         {
-                                                node.removeChild( eChildNode ) ;
+                                                // If the trimmed text node is empty, just remove it.
+                                                
+                                                // Use "eChildNode.parentNode" instead of "node" to avoid IE bug (#81).
+                                                eChildNode.parentNode.removeChild( eChildNode ) ;
                                                 continue ;
                                         }
                                         else if ( sTrimmed.length < iOriginalLength )
                                         {
+                                                // If the trimmed text lenght is less than the original
+                                                // lenght, strip all spaces from the end by splitting
+                                                // the text and removing the resulting useless node.
+
                                                 eChildNode.splitText( sTrimmed.length ) ;
-                                                node.removeChild( node.lastChild ) ;
+                                                // Use "node.lastChild.parentNode" instead of "node" to avoid IE bug (#81).
+                                                node.lastChild.parentNode.removeChild( node.lastChild ) ;
                                         }
                         }