Ticket #5404: 5404_2.patch

_source/plugins/htmldataprocessor/plugin.js

@@ -38 +38 @@
                 }
         }
 
-        function blockNeedsExtension( block )
+        function blockNeedsExtension( block, extendEmptyBlock )
         {
                 var lastChild = lastNoneSpaceChild( block );
 
-                return !lastChild
-                        || lastChild.type == CKEDITOR.NODE_ELEMENT && lastChild.name == 'br'
-                        // Some of the controls in form needs extension too,
-                        // to move cursor at the end of the form. (#4791)
-                        || block.name == 'form' && lastChild.name == 'input';
+                if( !extendEmptyBlock ||
+                        typeof extendEmptyBlock == 'function' && ( extendEmptyBlock( block ) == false ) )
+                        return false;
+
+                return !lastChild || lastChild &&
+                                ( lastChild.type == CKEDITOR.NODE_ELEMENT && lastChild.name == 'br'
+                                // Some of the controls in form needs extension too,
+                                // to move cursor at the end of the form. (#4791)
+                                || block.name == 'form' && lastChild.name == 'input' );
         }
 
-        function extendBlockForDisplay( block )
+        function getBlockExtension( isOutput, emptyBlockFiller )
         {
-                trimFillers( block, true );
-
-                if ( blockNeedsExtension( block ) )
+                return function( node )
                 {
-                        if ( CKEDITOR.env.ie )
-                                block.add( new CKEDITOR.htmlParser.text( '\xa0' ) );
-                        else
-                                block.add( new CKEDITOR.htmlParser.element( 'br', {} ) );
-                }
-        }
-
-        function extendBlockForOutput( block )
-        {
-                trimFillers( block );
-
-                if ( blockNeedsExtension( block ) )
-                        block.add( new CKEDITOR.htmlParser.text( '\xa0' ) );
+                        trimFillers( node, !isOutput );
+                        if ( blockNeedsExtension( node, !isOutput || emptyBlockFiller ) )
+                        {
+                                if ( isOutput || CKEDITOR.env.ie )
+                                        node.add( new CKEDITOR.htmlParser.text( '\xa0' ) );
+                                else
+                                        node.add( new CKEDITOR.htmlParser.element( 'br', {} ) );
+                        }
+                }
         }
 
         var dtd = CKEDITOR.dtd;
@@ -96 +94 @@
         var defaultDataBlockFilterRules = { elements : {} };
 
         for ( i in blockLikeTags )
-                defaultDataBlockFilterRules.elements[ i ] = extendBlockForDisplay;
+                defaultDataBlockFilterRules.elements[ i ] = getBlockExtension();
 
         var defaultHtmlFilterRules =
                 {
@@ -233 +231 @@
                         }
                 };
 
-        var defaultHtmlBlockFilterRules = { elements : {} };
-
-        for ( i in blockLikeTags )
-                defaultHtmlBlockFilterRules.elements[ i ] = extendBlockForOutput;
-
         if ( CKEDITOR.env.ie )
         {
                 // IE outputs style attribute in capital letters. We should convert
@@ -398 +391 @@
                         dataProcessor.dataFilter.addRules( defaultDataFilterRules );
                         dataProcessor.dataFilter.addRules( defaultDataBlockFilterRules );
                         dataProcessor.htmlFilter.addRules( defaultHtmlFilterRules );
+
+                        var defaultHtmlBlockFilterRules = { elements : {} };
+                        for ( i in blockLikeTags )
+                                defaultHtmlBlockFilterRules.elements[ i ] = getBlockExtension( true, editor.config.fillEmptyBlocks );
+
                         dataProcessor.htmlFilter.addRules( defaultHtmlBlockFilterRules );
                 }
         });
@@ -492 +490 @@
  * config.forceSimpleAmpersand = false;
  */
 CKEDITOR.config.forceSimpleAmpersand = false;
+
+/**
+ * Whether in the HTML output editor will insert a filler text (non-breaking space entity -  ) into empty block
+ * elements, in order to render such block element at an empty line height.
+ * @name CKEDITOR.config.fillEmptyBlocks;
+ * @type Boolean|Function a judging bool or a mediation function.
+ * @default true
+ * @example
+ * config.fillEmptyBlocks = false;      // Prevent filler nodes in all empty blocks.
+ *
+ * // Prevent filler node only in float cleaners.
+ * config.fillEmptyBlocks = function( element )
+ * {
+ *      if ( element.attributes[ 'class' ].indexOf ( 'clear-both' ) != -1 )
+ *              return false;
+ * }
+ */
+