return;\r
\r
this.range = range;\r
- this.forceBrBreak = false;\r
+ this.forceBrBreak = 0;\r
\r
// Whether include <br>s into the enlarged range.(#3730).\r
- this.enlargeBr = true;\r
- this.enforceRealBlocks = false;\r
+ this.enlargeBr = 1;\r
+ this.enforceRealBlocks = 0;\r
\r
this._ || ( this._ = {} );\r
}\r
// Indicats that the current element in the loop is the last one.\r
var isLast;\r
\r
+ // Indicate at least one of the range boundaries is inside a preformat block.\r
+ var touchPre;\r
+\r
// Instructs to cleanup remaining BRs.\r
var removePreviousBr, removeLastBr;\r
\r
// Shrink the range to exclude harmful "noises" (#4087, #4450, #5435).\r
range.shrink( CKEDITOR.NODE_ELEMENT, true );\r
\r
- range.enlarge( this.forceBrBreak || !this.enlargeBr ?\r
+ touchPre = range.endContainer.hasAscendant( 'pre', true )\r
+ || range.startContainer.hasAscendant( 'pre', true );\r
+\r
+ range.enlarge( this.forceBrBreak && !touchPre || !this.enlargeBr ?\r
CKEDITOR.ENLARGE_LIST_ITEM_CONTENTS : CKEDITOR.ENLARGE_BLOCK_CONTENTS );\r
\r
var walker = new CKEDITOR.dom.walker( range ),\r
// next block.(#3887)\r
if ( this._.lastNode &&\r
this._.lastNode.type == CKEDITOR.NODE_TEXT &&\r
- !CKEDITOR.tools.trim( this._.lastNode.getText( ) ) &&\r
+ !CKEDITOR.tools.trim( this._.lastNode.getText() ) &&\r
this._.lastNode.getParent().isBlockBoundary() )\r
{\r
var testRange = new CKEDITOR.dom.range( range.document );\r
{\r
// closeRange indicates that a paragraph boundary has been found,\r
// so the range can be closed.\r
- var closeRange = false;\r
+ var closeRange = 0,\r
+ parentPre = currentNode.hasAscendant( 'pre' );\r
\r
// includeNode indicates that the current node is good to be part\r
// of the range. By default, any non-element node is ok for it.\r
var includeNode = ( currentNode.type != CKEDITOR.NODE_ELEMENT ),\r
- continueFromSibling = false;\r
+ continueFromSibling = 0;\r
\r
// If it is an element node, let's check if it can be part of the\r
// range.\r
{\r
var nodeName = currentNode.getName();\r
\r
- if ( currentNode.isBlockBoundary( this.forceBrBreak && { br : 1 } ) )\r
+ if ( currentNode.isBlockBoundary( this.forceBrBreak &&\r
+ !parentPre && { br : 1 } ) )\r
{\r
// <br> boundaries must be part of the range. It will\r
// happen only if ForceBrBreak.\r
if ( nodeName == 'br' )\r
- includeNode = true;\r
+ includeNode = 1;\r
else if ( !range && !currentNode.getChildCount() && nodeName != 'hr' )\r
{\r
// If we have found an empty block, and haven't started\r
this._.nextNode = currentNode;\r
}\r
\r
- closeRange = true;\r
+ closeRange = 1;\r
}\r
else\r
{\r
currentNode = currentNode.getFirst();\r
continue;\r
}\r
- includeNode = true;\r
+ includeNode = 1;\r
}\r
}\r
else if ( currentNode.type == CKEDITOR.NODE_TEXT )\r
// Ignore normal whitespaces (i.e. not including or\r
// other unicode whitespaces) before/after a block node.\r
if ( beginWhitespaceRegex.test( currentNode.getText() ) )\r
- includeNode = false;\r
+ includeNode = 0;\r
}\r
\r
// The current node is good to be part of the range and we are\r
{\r
var parentNode = currentNode.getParent();\r
\r
- if ( parentNode.isBlockBoundary( this.forceBrBreak && { br : 1 } ) )\r
+ if ( parentNode.isBlockBoundary( this.forceBrBreak\r
+ && !parentPre && { br : 1 } ) )\r
{\r
- closeRange = true;\r
+ closeRange = 1;\r
isLast = isLast || ( parentNode.equals( lastNode) );\r
break;\r
}\r
\r
currentNode = parentNode;\r
- includeNode = true;\r
+ includeNode = 1;\r
isLast = ( currentNode.equals( lastNode ) );\r
- continueFromSibling = true;\r
+ continueFromSibling = 1;\r
}\r
}\r
\r
}\r
}\r
\r
+ // Ignore bookmark nodes.(#3783)\r
+ var bookmarkGuard = CKEDITOR.dom.walker.bookmark( false, true );\r
+\r
if ( removePreviousBr )\r
{\r
var previousSibling = block.getPrevious();\r
\r
if ( removeLastBr )\r
{\r
- // Ignore bookmark nodes.(#3783)\r
- var bookmarkGuard = CKEDITOR.dom.walker.bookmark( false, true );\r
-\r
var lastChild = block.getLast();\r
if ( lastChild && lastChild.type == CKEDITOR.NODE_ELEMENT && lastChild.getName() == 'br' )\r
{\r
block.getNextSourceNode( true, null, lastNode );\r
}\r
\r
+ if ( !bookmarkGuard( this._.nextNode ) )\r
+ {\r
+ this._.nextNode = this._.nextNode.getNextSourceNode( true, null, function( node )\r
+ { return !node.equals( lastNode ) && bookmarkGuard( node ); } );\r
+ }\r
+\r
return block;\r
}\r
};\r