Opened 14 years ago

Last modified 12 years ago

#6329 closed Bug

indent on list is not appropriate — at Version 2

Reported by: yiminghe Owned by:
Priority: Normal Milestone:
Component: Core : Lists Version: 3.4
Keywords: Cc:

Description (last modified by Tobiasz Cudnik)

1.type three sentence 2.outdent above line more than 3 times 3.click ul button to make above <p> to ul,li 4.inside the second li,click indent button

expected: see attachment expected.png

actual : see attachment 5.png

my solution:

pass li's attribute to its children, in list/plugin.js line 122:

for ( i = 0 ; i < item.contents.length ; i++ )
						currentListItem.append( item.contents[i].clone( true, true ) );

change to

for ( i = 0 ; i < item.contents.length ; i++ ){
						var ic=item.contents[i].clone( true, true );
                        if(currentListItem.type == CKEDITOR.NODE_DOCUMENT_FRAGMENT){
                            item.element.copyAttributes(ic);
                        }
                        currentListItem.append(ic);
                    }

Change History (8)

Changed 14 years ago by yiminghe

Attachment: 1.png added

Changed 14 years ago by yiminghe

Attachment: 2.png added

Changed 14 years ago by yiminghe

Attachment: 3.png added

Changed 14 years ago by yiminghe

Attachment: 4.png added

Changed 14 years ago by yiminghe

Attachment: 5.png added

Changed 14 years ago by yiminghe

Attachment: expected.png added

comment:1 Changed 14 years ago by yiminghe

Component: GeneralCore : Lists
Keywords: list indent added
Version: 3.4

comment:2 Changed 14 years ago by Tobiasz Cudnik

Description: modified (diff)
Keywords: list indent removed
Status: newpending

Why do you think we should not inherit indentation when creating a list ? (step 2 to 3)

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy