Ticket #6107: 6107_2.patch

_source/plugins/styles/plugin.js

@@ -149 +149 @@
                         return ( this.removeFromRange =
                                                 this.type == CKEDITOR.STYLE_INLINE ?
                                                         removeInlineStyle
+                                                : this.type == CKEDITOR.STYLE_BLOCK ?
+                                                        removeBlockStyle
                                                 : this.type == CKEDITOR.STYLE_OBJECT ?
                                                         removeObjectStyle
                                                 : null ).call( this, range );
@@ -881 +883 @@
                 range.moveToBookmark( bookmark );
         }
 
+        function removeBlockStyle( range )
+        {
+                // Serializible bookmarks is needed here since
+                // elements may be merged.
+                var bookmark = range.createBookmark( 1 );
+
+                var iterator = range.createIterator();
+                iterator.enforceRealBlocks = true;
+
+                iterator.enlargeBr = this._.enterMode != CKEDITOR.ENTER_BR;
+
+                var block;
+                var doc = range.document;
+
+                while ( ( block = iterator.getNextParagraph() ) )
+                {
+                        if ( !this.checkElementRemovable( block ) )
+                                continue;
+
+                        if ( this._.enterMode == CKEDITOR.ENTER_BR )  
+                                block.remove( 1 );  
+                        else  
+                                replaceBlock( block, new CKEDITOR.dom.element( this._.enterMode == CKEDITOR.ENTER_DIV ? 'div' : 'p' ) ); 
+                }
+
+                range.moveToBookmark( bookmark );
+        }
+
         // Replace the original block with new one, with special treatment
         // for <pre> blocks to make sure content format is well preserved, and merging/splitting adjacent
         // when necessary.(#3188)
@@ -1433 +1463 @@
 
                 var iterator = ranges.createIterator();
                 while ( ( range = iterator.getNextRange() ) )
-                {
                         func.call( this, range );
-                }
 
                 selection.selectRanges( ranges );
 

_source/plugins/stylescombo/plugin.js

@@ -105 +105 @@
                                                         selection = editor.getSelection();
 
                                                 var elementPath = new CKEDITOR.dom.elementPath( selection.getStartElement() );
+
+                                                style[ style.checkActive( elementPath ) ? 'remove' : 'apply' ]( editor.document );
 
-                                                if ( style.type == CKEDITOR.STYLE_INLINE && style.checkActive( elementPath ) )
-                                                        style.remove( editor.document );
-                                                else if ( style.type == CKEDITOR.STYLE_OBJECT && style.checkActive( elementPath ) )
-                                                        style.remove( editor.document );
-                                                else
-                                                        style.apply( editor.document );
-
                                                 editor.fire( 'saveSnapshot' );
                                         },