\r
var selectAllCmd =\r
{\r
+ modes : { wysiwyg : 1, source : 1 },\r
exec : function( editor )\r
{\r
switch ( editor.mode )\r
editor.document.$.execCommand( 'SelectAll', false, null );\r
break;\r
case 'source' :\r
- // TODO\r
+ // Select the contents of the textarea\r
+ var textarea = editor.textarea.$ ;\r
+ if ( CKEDITOR.env.ie )\r
+ {\r
+ textarea.createTextRange().execCommand( 'SelectAll' ) ;\r
+ }\r
+ else\r
+ {\r
+ textarea.selectionStart = 0 ;\r
+ textarea.selectionEnd = textarea.value.length ;\r
+ }\r
+ textarea.focus() ;\r
}\r
},\r
canUndo : false\r
// "onfocusin" is fired before "onfocus". It makes it\r
// possible to restore the selection before click\r
// events get executed.\r
- body.on( 'focusin', function()\r
+ body.on( 'focusin', function( evt )\r
{\r
+ // If there are elements with layout they fire this event but\r
+ // it must be ignored to allow edit its contents #4682\r
+ if ( evt.data.$.srcElement.nodeName != 'BODY' )\r
+ return;\r
+\r
// If we have saved a range, restore it at this\r
// point.\r
if ( savedRange )\r
}\r
});\r
\r
- editor.window.on( 'focus', function()\r
+ body.on( 'focus', function()\r
{\r
// Enable selections to be saved.\r
saveEnabled = true;\r
saveSelection();\r
});\r
\r
- body.on( 'beforedeactivate', function()\r
+ body.on( 'beforedeactivate', function( evt )\r
{\r
+ // Ignore this event if it's caused by focus switch between\r
+ // internal editable control type elements, e.g. layouted paragraph. (#4682)\r
+ if ( evt.data.$.toElement )\r
+ return;\r
+\r
// Disable selections from being saved.\r
saveEnabled = false;\r
});\r
\r
+ // IE before version 8 will leave cursor blinking inside the document after\r
+ // editor blurred unless we clean up the selection. (#4716)\r
+ if ( CKEDITOR.env.ie && CKEDITOR.env.version < 8 )\r
+ {\r
+ doc.getWindow().on( 'blur', function( evt )\r
+ {\r
+ editor.document.$.selection.empty();\r
+ });\r
+ }\r
+\r
// IE fires the "selectionchange" event when clicking\r
// inside a selection. We don't want to capture that.\r
body.on( 'mousedown', disableSave );\r
testRange = range.duplicate();\r
\r
testRange.moveToElementText( child );\r
- testRange.collapse();\r
\r
- var comparison = testRange.compareEndPoints( 'StartToStart', range );\r
+ var comparisonStart = testRange.compareEndPoints( 'StartToStart', range ),\r
+ comparisonEnd = testRange.compareEndPoints( 'EndToStart', range );\r
+\r
+ testRange.collapse();\r
\r
- if ( comparison > 0 )\r
+ if ( comparisonStart > 0 )\r
break;\r
- else if ( comparison === 0 )\r
- return {\r
- container : parent,\r
- offset : i\r
- };\r
+ // When selection stay at the side of certain self-closing elements, e.g. BR,\r
+ // our comparison will never shows an equality. (#4824)\r
+ else if ( !comparisonStart\r
+ || comparisonEnd == 1 && comparisonStart == -1 )\r
+ return { container : parent, offset : i };\r
+ else if ( !comparisonEnd )\r
+ return { container : parent, offset : i + 1 };\r
\r
testRange = null;\r
}\r
// breaking character counting logic below. (#3949)\r
var distance = testRange.text.replace( /(\r\n|\r)/g, '\n' ).length;\r
\r
- while ( distance > 0 )\r
- distance -= siblings[ --i ].nodeValue.length;\r
+ try\r
+ {\r
+ while ( distance > 0 )\r
+ distance -= siblings[ --i ].nodeValue.length;\r
+ }\r
+ // Measurement in IE could be somtimes wrong because of <select> element. (#4611)\r
+ catch( e )\r
+ {\r
+ distance = 0;\r
+ }\r
+\r
\r
if ( distance === 0 )\r
{\r
// Decrease the range content to exclude particial\r
// selected node on the start which doesn't have\r
// visual impact. ( #3231 )\r
- while( true )\r
+ while ( true )\r
{\r
var startContainer = range.startContainer,\r
startOffset = range.startOffset;\r
if ( cache.selectedElement !== undefined )\r
return cache.selectedElement;\r
\r
- var node;\r
+ var self = this;\r
\r
- if ( this.getType() == CKEDITOR.SELECTION_ELEMENT )\r
- {\r
- var sel = this.getNative();\r
-\r
- if ( CKEDITOR.env.ie )\r
+ var node = CKEDITOR.tools.tryThese(\r
+ // Is it native IE control type selection?\r
+ function()\r
{\r
- try\r
+ return self.getNative().createRange().item( 0 );\r
+ },\r
+ // Figure it out by checking if there's a single enclosed\r
+ // node of the range.\r
+ function()\r
+ {\r
+ var range = self.getRanges()[ 0 ];\r
+ range.shrink( CKEDITOR.SHRINK_ELEMENT );\r
+\r
+ var enclosed;\r
+ if ( range.startContainer.equals( range.endContainer )\r
+ && ( range.endOffset - range.startOffset ) == 1\r
+ && styleObjectElements[ ( enclosed = range.startContainer.getChild( range.startOffset ) ).getName() ] )\r
{\r
- node = sel.createRange().item(0);\r
+ return enclosed.$;\r
}\r
- catch(e) {}\r
- }\r
- else\r
- {\r
- var range = sel.getRangeAt( 0 );\r
- node = range.startContainer.childNodes[ range.startOffset ];\r
- }\r
- }\r
+ });\r
\r
return cache.selectedElement = ( node ? new CKEDITOR.dom.element( node ) : null );\r
},\r
range.moveToElementText( element.$ );\r
range.select();\r
}\r
+ finally\r
+ {\r
+ this.document.fire( 'selectionchange' );\r
+ }\r
\r
this.reset();\r
}\r
else\r
ieRange.select();\r
\r
+ this.moveToPosition( dummySpan, CKEDITOR.POSITION_BEFORE_START );\r
dummySpan.remove();\r
}\r
else\r
endNode.remove();\r
ieRange.select();\r
}\r
+\r
+ this.document.fire( 'selectionchange' );\r
}\r
:\r
function()\r
Got questions, comments, patches, etc.? Contact Jason Woofenden