Changeset 6182

Timestamp:

12/07/10 13:44:22 (2 years ago)

Author:

garry.yao

Message:

#6779: Select body contents instead of selecting body node.

Location:

CKEditor/trunk

Files:

• CHANGES.html (modified) (1 diff)
• _source/plugins/elementspath/plugin.js (modified) (1 diff)

CKEditor/trunk/CHANGES.html

@@ -68 +68 @@
                 <li><a href="http://dev.ckeditor.com/ticket/6150">#6150</a> : [Safari] Color dialog was broken.</li>
                 <li><a href="http://dev.ckeditor.com/ticket/6747">#6747</a> : Full screen layout issue caused by page element focus outside editor.</li>
+                <li><a href="http://dev.ckeditor.com/ticket/6779">#6779</a> : Click on "body" on elements blink the selection off.</li>
                 <li>Updated the following language files:<ul>
                         <li><a href="http://dev.ckeditor.com/ticket/6756">#6756</a> : Hungarian;</li>

CKEditor/trunk/_source/plugins/elementspath/plugin.js

@@ -156 +156 @@
 
                 var element = editor._.elementsPath.list[ elementIndex ];
-                editor.getSelection().selectElement( element );
+
+                if ( element.is( 'body' ) )
+                {
+                        var range = new CKEDITOR.dom.range( editor.document );
+                        range.selectNodeContents( element );
+                        range.select();
+                }
+                else
+                        editor.getSelection().selectElement( element );
 
                 return false;