Problems when pasting lists from word

[Jakub Ś]

This issue was found when reproducing #7982

1. Paste the list from the attached file to CKEditor
2. The last second-level list items will be changed to first-level list items.

The file contains description how this list was created in WORD.

Pasting such lists worked till version 3.6.
From revision [6912] it got broken. All elements were pasted as first level list-items.
In rev [6977] there was a slight improvement. Items after first sub-list weren't pasted.
From rev [7008] it works as described above.

[Jakub Ś]

#8450 was marked as duplicate

[Jakub Ś]

Possible duplicate was reported in #8754

Revisions are the same but the issue is reproducible in Firefox, when copying/pasting HTML and only with PasteFromWord.

[jorp]

I finally got around to upgrading from 3.6 to 3.6.6.1 and unfortunately this is still an issue.

I dug into the code -- plugins/pastefromword/filter/default.js -- and I think the following is the problem:

In the flattenList() function, the variable children is set to element.children. When a sublist is processed, children gets set to an array. On the next for-loop iteration, the children variable does not have the expected values.

The fix is to reset children to element.children.

var children = element.children,
    child;

for ( var i = 0; i < children.length; i++ )
{
    child = children[ i ];
    
    if ( child.name in CKEDITOR.dtd.$listItem )
    {
        ...
    }
    // Flatten sub list.
    else if ( child.name in CKEDITOR.dtd.$list )
    {
        // Absorb sub list children.
        arguments.callee.apply( this, [ child, level + 1 ] );
        children = children.slice( 0, i ).concat( child.children ).concat( children.slice( i + 1 ) );
        element.children = [];
        for ( var j = 0, num = children.length; j < num ; j++ )
            element.add( children[ j ] );

        // BUGFIX: above, children set to an array, reset to reference element.children
        children = element.children;
    }
}

I hope that helps.

[Jakub Ś]

#10672 was marked as duplicate.

[Frederico Caldeira Knabben]

Thanks jorp. We gonna check this.

I'm tentatively targeting this ticket to the next release.

[Teresa Fairchild]

I am having this exact same issue as well. I hope it gets resolved soon, because my users will be creating many outlines and lists.

[Piotrek Koszuliński]

Pushed ​t/8103 on dev and tests.

Note for a reviewer: 3rd level of lists indetation does not work (at least on Webkit), but this is a different issue, caused by the fact that 3rd level is hoisted by browser/MSWord from list structure to normal paragraphs.

@jorp: Thanks for this patch.

[Jakub Ś]

I have checked branch t/8103 and it seems to work fine.

The only thing that doesn't work are these 3-rd level lists on webkit (although Safari 5.1 wors as expected).

[Piotrek Koszuliński]

Fixed on master with ​git:acaffaa and a21a00a on tests.

[nikhil]

i am using CKeditor 4.4,4 but still i am facing the same issue please help us to resolve it

[Jakub Ś]

I have checked this in Webkit and Blink browsers on Windows and Mac but everything seems to be fine. I have tested this issue with lists in attachment.

1. Please attach docx file with list that is causing problems for you
2. Please tell in which browser and which Operating System does it happen
3. Perhaps you are using some third-party plugins that influence this behaviour? Does it happen on demo page or with standard/full pre-set downloaded from our page?