Ticket #3165: 3165.patch

_source/plugins/enterkey/plugin.js

@@ -101 +101 @@
                 if ( !isStartOfBlock && !isEndOfBlock )
                 {
                         // If the next block is an <li> with another list tree as the first
-                        // child, we'll need to append a placeholder or the list item
+                        // child, we'll need to append a filler <br> or the list item
                         // wouldn't be editable. (#1420)
-                        if ( nextBlock.is( 'li' ) && ( node = nextBlock.getFirst() )
-                                        && node.is && node.is( 'ul', 'ol') )
-                                nextBlock.insertBefore( doc.createText( '\xa0' ), node );
+                        if ( nextBlock.is( 'li' )
+                                 && ( node = nextBlock.getFirst( CKEDITOR.dom.walker.invisible( true ) ) )
+                                 && node.is && node.is( 'ul', 'ol' ) )
+                                doc.createElement( 'br' ).insertBefore( node );
 
                         // Move the selection to the end block.
                         if ( nextBlock )

_source/core/dom/element.js

@@ -627 +627 @@
 
                 /**
                  * Gets the first child node of this element.
+                 * @param {Function} evaluator Filtering the result node.
                  * @returns {CKEDITOR.dom.node} The first child node or null if not
                  *              available.
                  * @example
@@ -634 +635 @@
                  * var first = <b>element.getFirst()</b>;
                  * alert( first.getName() );  // "b"
                  */
-                getFirst : function()
+                getFirst : function( evaluator )
                 {
-                        var $ = this.$.firstChild;
-                        return $ ? new CKEDITOR.dom.node( $ ) : null;
+                        var first = this.$.firstChild,
+                                retval = first && new CKEDITOR.dom.node( first );
+                        if ( retval && evaluator && !evaluator( retval ) )
+                                retval = retval.getNext( evaluator );
+
+                        return retval;
                 },
 
                 /**

_source/core/dom/walker.js

@@ -408 +408 @@
                         return isReject ^ isWhitespace;
                 };
         };
+
+        /**
+         * Whether the node is invisible in wysiwyg mode.
+         * @param isReject
+         */
+        CKEDITOR.dom.walker.invisible = function( isReject )
+        {
+                var whitespace = CKEDITOR.dom.walker.whitespaces();
+                return function( node )
+                {
+                        // Nodes that take no spaces in wysiwyg:
+                        // 1. White-spaces but not including NBSP;
+                        // 2. Empty inline elements, e.g. <b></b> we're checking here 'offsetHeight'
+                        // instead of width for properly excluding all sorts of empty paragraph, e.g. <br />.
+                        var isInvisible = whitespace( node )
+                                        || !node.$.offsetHeight;
+                        return isReject ^ isInvisible;
+                };
+        };
+
 })();