Changeset 6413

Timestamp:

02/09/11 15:33:33 (2 years ago)

Author:

Saare

Message:

#7102: Divreplace sample sometimes does not work.

Location:

CKEditor/trunk

Files:

• CHANGES.html (modified) (1 diff)
• _samples/divreplace.html (modified) (1 diff)

CKEditor/trunk/CHANGES.html

@@ -46 +46 @@
         <ul>
                 <li><a href="http://dev.ckeditor.com/ticket/7054">#7054</a> : Special characters' tooltips are now lowercased, making them more readable.</li>
+                <li><a href="http://dev.ckeditor.com/ticket/7102">#7102</a> : "Replacing Div" sample didn't work when double clicking inside of formatted text.</li>
                 <li>Updated the following language files:<ul>
                         <li><a href="http://dev.ckeditor.com/ticket/7124">#7124</a> : Czech;</li>

CKEditor/trunk/_samples/divreplace.html

@@ -53 +53 @@
 
         // Find out the div that holds this element.
-        element = element.parentNode;
+        var name;
+        do
+        {
+                element = element.parentNode;
+        }
+        while ( element && ( name = element.nodeName.toLowerCase() ) && ( name != 'div' || element.className.indexOf( 'editable' ) == -1 ) && name != 'body' )
+        
 
-        if ( element.nodeName.toLowerCase() == 'div'
-                 && ( element.className.indexOf( 'editable' ) != -1 ) )
+        if ( name == 'div' && element.className.indexOf( 'editable' ) != -1 )
                 replaceDiv( element );
 }