Ticket #3509: 3509_2.patch

_source/plugins/enterkey/plugin.js

@@ -139 +139 @@
                         if ( !newBlock )
                                 newBlock = doc.createElement( blockTag );
 
+                        // We need to insert the bogus element inside all of the inline
+                        // style elements, so that clicking after the line or pressing end
+                        // and typing makes the typed text keep these inline styles.
+                        var blockWithBogus = newBlock;
+
                         // Recreate the inline elements tree, which was available
                         // before hitting enter, so the same styles will be available in
                         // the new block.
@@ -157 +162 @@
                                                 element = element.clone();
                                                 newBlock.moveChildren( element );
                                                 newBlock.append( element );
+
+                                                blockWithBogus = element;
                                         }
                                 }
                         }
 
                         if ( !CKEDITOR.env.ie )
-                                newBlock.appendBogus();
+                                blockWithBogus.appendBogus();
 
                         range.insertNode( newBlock );
 

_source/core/dom/element.js

@@ -243 +243 @@
                                 lastChild = lastChild.getPrevious();
                         if ( !lastChild || !lastChild.is || !lastChild.is( 'br' ) )
                         {
-                                this.append(
-                                        CKEDITOR.env.opera ?
-                                                this.getDocument().createText('') :
-                                                this.getDocument().createElement( 'br' ) );
+                                var bogusElement = CKEDITOR.env.opera ?
+                                                this.getDocument().createText( '' ) :
+                                                this.getDocument().createElement( 'br' );
+                                this.append( bogusElement );
+                                return bogusElement;
                         }
                 },
 

_source/core/dom/range.js

@@ -1299 +1299 @@
 
                         this.collapse( isStart );
 
+                        // If endContainer is at the end of the document, we need to create
+                        // a bogus element so that the range can include the entire endContainer.
+                        // (This isn't necessary otherwise, but doesn't hurt.)
+                        var tempBogus = null;
+                        if ( !CKEDITOR.env.ie )
+                                tempBogus = this.endContainer.getParent().appendBogus();
+
                         this.enlarge( CKEDITOR.ENLARGE_BLOCK_CONTENTS );
 
                         this.extractContents().appendTo( fixedBlock );
@@ -1307 +1314 @@
                         if ( !CKEDITOR.env.ie )
                                 fixedBlock.appendBogus();
 
+                        if ( tempBogus )
+                                tempBogus.remove();
+
                         this.insertNode( fixedBlock );
 
                         this.moveToBookmark( bookmark );