Ticket #8249 (closed Bug: fixed)
[FF]: Problems with deleting last list item
| Reported by: | j.swiderski | Owned by: | garry.yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.6.3 |
| Component: | Core : Lists | Version: | 3.0 |
| Keywords: | IBM | Cc: | satya_minnekanti@… |
Description (last modified by j.swiderski) (diff)
I have found this issue when reproducing 8248
Scenario 1
- Paste the following code:
<ol> <li> FirstListItem </li> <li> SecondListItem <ol> <li> ThirdListItem </li> <li> FourthListItem </li> <li> FifthListItem </li> </ol> </li> <li> SixthListItem </li> </ol>
- Place the cursor before "sixth item" ^SixthListItem and press backspace
Result: You wont be able to delete "3".
NOTE: Webkit and Opera concatenates fith item with sixth item - FifthListItemSixthListItem. IE puts sixth item in new paragraph. Not sure which behavior is correct.
Scenario 2
- The same as above
- Put the cursor behind fifth item FifthListItem^ and start pressing delete.
Result: When all text in sixth item gets deleted, number is deleted at the end.
Attachments
Change History
comment:5 Changed 18 months ago by garry.yao
- Owner set to garry.yao
- Status changed from confirmed to review
Normalize this specific backspace to MS-Word alike behavior which outdent the list item.
comment:6 Changed 15 months ago by fredck
- Status changed from review to review_failed
The idea is good and it works well.
But just to avoid people complaining later, let's have it working well for this case:
- Load this HTML and selection:
<ul> <li> <p>Para 1</p> <p>^Para 2</p> </li> </ul>
- BACKSPACE
After patch, the li will outdent, instead of having the paragraphs merged.
comment:7 Changed 14 months ago by garry.yao
- Status changed from review_failed to review
- Keywords Firefox removed
comment:8 Changed 14 months ago by fredck
- Status changed from review to review_passed
- Milestone set to CKEditor 3.6.3
comment:9 Changed 14 months ago by garry.yao
- Status changed from review_passed to closed
- Resolution set to fixed
Fixed with [7392].
comment:10 Changed 14 months ago by garry.yao
One more new ticket test failed, post fixed with [7424].
