Opened 14 years ago
Closed 13 years ago
#6329 closed Bug (fixed)
indent on list is not appropriate
Reported by: | yiminghe | Owned by: | |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | Core : Lists | Version: | 3.4 |
Keywords: | Cc: |
Description (last modified by )
- type three sentence
- outdent above line more than 3 times
- click ul button to make above <p> to ul,li
- 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); }
Attachments (6)
Change History (10)
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 |
comment:3 Changed 14 years ago by
Description: | modified (diff) |
---|
comment:4 Changed 13 years ago by
Resolution: | → fixed |
---|---|
Status: | pending → closed |
I have checked the issue in latest CKEditor 3.6.2.
It seems that lists are working now just like @yiminghe expected
Why do you think we should not inherit indentation when creating a list ? (step 2 to 3)