Opened 9 years ago
Closed 9 years ago
#6236 closed Bug (fixed)
[[IE]] Issues with Numbered list
Reported by: | Satya Minnekanti | Owned by: | Garry Yao |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.5.3 |
Component: | Core : Lists | Version: | |
Keywords: | IBM | Cc: | Damian, joek |
Description
To reproduce the defect:
- Open Ajax sample.
- Create a 2 level numbered/bulleted list.
- use back space to delete the list up until you reach the last item in the 2nd level
- Press enter 3 times to come out of the list.
Expected Result:
Cursor comes out of the list and a new empty paragraph is created.
Actual Result:
Cursor comes out of the list but it is removing the list items in level 2.
I am attaching screenshots of different steps of the issue
Attachments (5)
Change History (15)
Changed 9 years ago by
Attachment: | Initial list.jpg added |
---|
Changed 9 years ago by
Attachment: | deletting list up to last item in 2nd level.jpg added |
---|
Changed 9 years ago by
Attachment: | list after pressing enter 3 times.jpg added |
---|
comment:1 Changed 9 years ago by
Component: | General → Core : Lists |
---|---|
Status: | new → confirmed |
comment:2 Changed 9 years ago by
Owner: | set to Paweł Horzela |
---|---|
Status: | confirmed → assigned |
comment:3 Changed 9 years ago by
Status: | assigned → review |
---|
comment:4 Changed 9 years ago by
comment:5 Changed 9 years ago by
Milestone: | → CKEditor 3.5.2 |
---|
comment:6 Changed 9 years ago by
Simplified TC:
- Load the editor with the following content and selection:
<ol> <li> item1 <ol> <li> item2</li> </ol> </li> </ol> <p> </p>
- Put cursor after list (in blank paragraph) and press "Backspace";
- Actual Result: Note there're two consequent "ol" in elements path bar.
comment:7 Changed 9 years ago by
Status: | review → review_failed |
---|
Though the root problem is the bug brought by "Backspace" which is browser intrinsic, Pawel's work around does works well as we have also similar fix in the html parser, so editor is able to output correct data with even a wrong DOM structure.
I'd still give R- to it as the fix don't have to be so specific.
Changed 9 years ago by
Attachment: | 6236_2.patch added |
---|
comment:8 Changed 9 years ago by
Owner: | changed from Paweł Horzela to Garry Yao |
---|---|
Status: | review_failed → review |
comment:9 Changed 9 years ago by
Status: | review → review_passed |
---|
comment:10 Changed 9 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Fixed with [6558].
This issue is connected with issue 6159 so remember to merge #6159.patch with #6236.patch.