Changeset 297

Timestamp:

2007-05-09 23:07:15 (3 years ago)

Author:

alfonsoml

Message:

Fix for #324, IE doesn't parse elements to a true DOM tree so a child might not have the parent that you expect!

Location:

FCKeditor/trunk

Files:

• editor/_source/internals/fckdomtools.js (modified) (1 diff)
• _whatsnew.html (modified) (1 diff)

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

@@ -78 +78 @@
                                         if ( eChildNode.nodeName.toUpperCase() == 'BR' && ( ignoreEndBRs || eChildNode.getAttribute( 'type', 2 ) == '_moz' ) )
                                         {
-                                                node.removeChild( eChildNode ) ;
+                                                // Use "eChildNode.parentNode" instead of "node" to avoid IE bug (#324).
+                                                eChildNode.parentNode.removeChild( eChildNode ) ;
                                                 continue ;
                                         }

FCKeditor/trunk/_whatsnew.html

@@ -78 +78 @@
                 <li>[<a target="_blank" href="https://sourceforge.net/tracker/?func=detail&aid=1518766&group_id=75348&atid=543653">SF
                         BUG-1518766</a>] Mozilla 1.7.13 wasn't recognized properly as an old Gecko engine.</li>
+                <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/324">#324</a>] Improperly nested
+                        tags could lead to a crash in IE.</li>
         </ul>
         <h3>