Opened 14 years ago

Closed 13 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:

  1. Open Ajax sample.
  1. Create a 2 level numbered/bulleted list.
  1. use back space to delete the list up until you reach the last item in the 2nd level
  1. 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)

Initial list.jpg (32.1 KB) - added by Satya Minnekanti 14 years ago.
deletting list up to last item in 2nd level.jpg (42.5 KB) - added by Satya Minnekanti 14 years ago.
list after pressing enter 3 times.jpg (31.4 KB) - added by Satya Minnekanti 14 years ago.
6236.patch (675 bytes) - added by Paweł Horzela 14 years ago.
Patch
6236_2.patch (770 bytes) - added by Garry Yao 13 years ago.

Download all attachments as: .zip

Change History (15)

Changed 14 years ago by Satya Minnekanti

Attachment: Initial list.jpg added

Changed 14 years ago by Satya Minnekanti

Changed 14 years ago by Satya Minnekanti

comment:1 Changed 14 years ago by Sa'ar Zac Elias

Component: GeneralCore : Lists
Status: newconfirmed

comment:2 Changed 14 years ago by Paweł Horzela

Owner: set to Paweł Horzela
Status: confirmedassigned

comment:3 Changed 14 years ago by Paweł Horzela

Status: assignedreview

comment:4 Changed 14 years ago by Paweł Horzela

This issue is connected with issue 6159 so remember to merge #6159.patch with #6236.patch.

Changed 14 years ago by Paweł Horzela

Attachment: 6236.patch added

Patch

comment:5 Changed 13 years ago by Wiktor Walc

Milestone: CKEditor 3.5.2

comment:6 Changed 13 years ago by Garry Yao

Simplified TC:

  1. Load the editor with the following content and selection:
    <ol>
    	<li>
    		item1
    		<ol>
    			<li>
    				item2</li>
    		</ol>
    	</li>
    </ol>
    <p>&nbsp;</p>
    
  2. 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 13 years ago by Garry Yao

Status: reviewreview_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 13 years ago by Garry Yao

Attachment: 6236_2.patch added

comment:8 Changed 13 years ago by Garry Yao

Owner: changed from Paweł Horzela to Garry Yao
Status: review_failedreview

comment:9 Changed 13 years ago by Sa'ar Zac Elias

Status: reviewreview_passed

comment:10 Changed 13 years ago by Garry Yao

Resolution: fixed
Status: review_passedclosed

Fixed with [6558].

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy