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 )
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
Changed 14 years ago by
Changed 14 years ago by
Changed 14 years ago by
Changed 14 years ago by
Changed 14 years ago by
Attachment: | expected.png added |
---|
comment:1 Changed 14 years ago by
Component: | General → Core : Lists |
---|---|
Keywords: | list indent added |
Version: | → 3.4 |
comment:2 Changed 14 years ago by
Description: | modified (diff) |
---|---|
Keywords: | list indent removed |
Status: | new → pending |
Why do you think we should not inherit indentation when creating a list ? (step 2 to 3)