Ticket #6662: 6662.patch

_source/plugins/pastefromword/filter/default.js

@@ -184 +184 @@
                         isListBulletIndicator : function( element )
                         {
                                 var styleText = element.attributes && element.attributes.style;
-                                if ( /mso-list\s*:\s*Ignore/i.test( styleText ) )
+                                if ( !CKEDITOR.env.gecko && /mso-list\s*:\s*Ignore/i.test( styleText ) )
                                         return true;
+                                // Search for following structure in Gecko `P.class > SPAN > SPAN > SPAN`.
+                                else if ( CKEDITOR.env.gecko && element.parent.attributes && 
+                                        /\b(ListParagraph(CxSp\w+)?|MsoNormal\b)/i.test( element.parent.attributes[ 'class' ] ) &&
+                                        element.children && element.children.length == 1 && element.children[ 0 ].name == 'span' &&
+                                        element.children[ 0 ].children && element.children[ 0 ].children.length == 1 &&
+                                        element.children[ 0 ].children[ 0 ].type == 3
+                                )
+                                        return true;
                         },
 
                         isContainingOnlySpaces : function( element )
@@ -214 +222 @@
                                                                         // Text-indent is not representing list item level any more.
                                                                         [ 'text-indent' ],
                                                                         [ 'line-height' ],
+
+                                                                        [ 'list-style-type' ],
                                                                         // Resolve indent level from 'margin-left' value.
                                                                         [ ( /^margin(:?-left)?$/ ), null, function( margin )
                                                                         {
@@ -399 +409 @@
                                                                         ] )( listItemAttrs.style )
                                                                         || '' );
 
-                                                        if ( !list )
+                                                        if ( !list || ( list && list.name != listType ) )
                                                         {
                                                                 list = new CKEDITOR.htmlParser.element( listType );
                                                                 list.add( listItem );
@@ -867 +877 @@
 
                                                 // For IE/Safari: List item bullet type is supposed to be indicated by
                                                 // the text of a span with style 'mso-list : Ignore' or an image.
-                                                if ( !CKEDITOR.env.gecko && isListBulletIndicator( element ) )
+                                                // For Gecko, list bullet is a child of `p.ListParagraphCxSp*` element.
+                                                if ( ( !CKEDITOR.env.gecko && isListBulletIndicator( element ) ) ||
+                                                        ( CKEDITOR.env.gecko && isListBulletIndicator( element ) ) )
                                                 {
-                                                        var listSymbolNode = element.firstChild( function( node )
-                                                        {
-                                                                return node.value || node.name == 'img';
-                                                        });
-
-                                                        var listSymbol =  listSymbolNode && ( listSymbolNode.value || 'l.' ),
-                                                                listType = listSymbol.match( /^([^\s]+?)([.)]?)$/ );
+                                                        var listSymbol, listType;
+                                                        if ( CKEDITOR.env.gecko )
+                                                        {
+                                                                listType = element.children[ 0 ].children[ 0 ].value;
+                                                                listType = /^(?: [ \n]?)*([\w\u00B7]*)(\.?)/.exec( listType );
+                                                        }
+                                                        else
+                                                        {
+                                                                var listSymbolNode = element.firstChild( function( node )
+                                                                {
+                                                                        return node.value || node.name == 'img';
+                                                                });
+                                                                
+                                                                listSymbol =  listSymbolNode && ( listSymbolNode.value || 'l.' ),
+                                                                        listType = listSymbol.match( /^([^\s]+?)([.)]?)$/ );
+                                                        }
                                                         return createListBulletMarker( listType, listSymbol );
                                                 }
 
@@ -971 +992 @@
                                                 // Preserve clear float style.
                                                 [ ( /^clear$/ ) ],
 
+                                                [ ( /^list-style-type/ ) ],
+
                                                 [ ( /^border.*|margin.*|vertical-align|float$/ ), null,
                                                         function( value, element )
                                                         {