Opened 14 years ago
Closed 14 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 14 years ago by
Attachment: | Initial list.jpg added |
---|
Changed 14 years ago by
Attachment: | deletting list up to last item in 2nd level.jpg added |
---|
Changed 14 years ago by
Attachment: | list after pressing enter 3 times.jpg added |
---|
comment:1 Changed 14 years ago by
Component: | General → Core : Lists |
---|---|
Status: | new → confirmed |
comment:2 Changed 14 years ago by
Owner: | set to Paweł Horzela |
---|---|
Status: | confirmed → assigned |
comment:3 Changed 14 years ago by
Status: | assigned → review |
---|
comment:4 Changed 14 years ago by
comment:5 Changed 14 years ago by
Milestone: | → CKEditor 3.5.2 |
---|
comment:6 Changed 14 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 14 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 14 years ago by
Attachment: | 6236_2.patch added |
---|
comment:8 Changed 14 years ago by
Owner: | changed from Paweł Horzela to Garry Yao |
---|---|
Status: | review_failed → review |
comment:9 Changed 14 years ago by
Status: | review → review_passed |
---|
comment:10 Changed 14 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.