Opened 15 years ago
Closed 14 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 15 years ago by
Changed 15 years ago by
Changed 15 years ago by
Changed 15 years ago by
Changed 15 years ago by
Changed 15 years ago by
| Attachment: | expected.png added |
|---|
comment:1 Changed 15 years ago by
| Component: | General → Core : Lists |
|---|---|
| Keywords: | list indent added |
| Version: | → 3.4 |
comment:2 Changed 15 years ago by
| Description: | modified (diff) |
|---|---|
| Keywords: | list indent removed |
| Status: | new → pending |
comment:3 Changed 15 years ago by
| Description: | modified (diff) |
|---|
comment:4 Changed 14 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)