Changeset 7508


Ignore:
Timestamp:
06/21/12 18:12:25 (3 years ago)
Author:
garry.yao
Message:

#8997: Fixed style not copied from the list root when converting.

Location:
CKEditor/trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • CKEditor/trunk/CHANGES.html

    r7507 r7508  
    5656                <li><a href="http://dev.ckeditor.com/ticket/8979">#8979</a> : Overriding matches will not any more interfere on the Font size combo.</li>
    5757                <li><a href="http://dev.ckeditor.com/ticket/5447">#5447</a> : Trimmed anchor name on dialog field.</li>
     58                <li><a href="http://dev.ckeditor.com/ticket/8997">#8997</a> : Styles are lost on sub-lists when convering list type.</li>
    5859        </ul>
    5960        <h3>
  • CKEditor/trunk/_source/plugins/list/plugin.js

    r7497 r7508  
    300300
    301301                var root = groupObj.root,
    302                         fakeParent = root.getDocument().createElement( this.type );
    303                 // Copy all attributes, except from 'start' and 'type'.
    304                 root.copyAttributes( fakeParent, { start : 1, type : 1 } );
    305                 // The list-style-type property should be ignored.
    306                 fakeParent.removeStyle( 'list-style-type' );
     302                        doc = root.getDocument(),
     303                        listNode,
     304                        newListNode;
    307305
    308306                for ( i = 0 ; i < selectedListItems.length ; i++ )
    309307                {
    310308                        var listIndex = selectedListItems[i].getCustomData( 'listarray_index' );
    311                         listArray[listIndex].parent = fakeParent;
    312                 }
     309                        listNode = listArray[ listIndex ].parent;
     310
     311                        // Switch to new list node for this particular item.
     312                        if ( !listNode.is( this.type ) )
     313                        {
     314                                newListNode = doc.createElement( this.type );
     315                                // Copy all attributes, except from 'start' and 'type'.
     316                                listNode.copyAttributes( newListNode, { start : 1, type : 1 } );
     317                                // The list-style-type property should be ignored.
     318                                newListNode.removeStyle( 'list-style-type' );
     319                                listArray[ listIndex ].parent = newListNode;
     320                        }
     321                }
     322
    313323                var newList = CKEDITOR.plugins.list.arrayToList( listArray, database, null, editor.config.enterMode );
    314324                var child, length = newList.listNode.getChildCount();
Note: See TracChangeset for help on using the changeset viewer.
© 2003 – 2015 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy