Opened 7 years ago

Closed 7 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 7 years ago.
deletting list up to last item in 2nd level.jpg (42.5 KB) - added by Satya Minnekanti 7 years ago.
list after pressing enter 3 times.jpg (31.4 KB) - added by Satya Minnekanti 7 years ago.
6236.patch (675 bytes) - added by Paweł Horzela 7 years ago.
Patch
6236_2.patch (770 bytes) - added by Garry Yao 7 years ago.

Download all attachments as: .zip

Change History (15)

Changed 7 years ago by Satya Minnekanti

Attachment: Initial list.jpg added

Changed 7 years ago by Satya Minnekanti

Changed 7 years ago by Satya Minnekanti

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

Component: GeneralCore : Lists
Status: newconfirmed

comment:2 Changed 7 years ago by Paweł Horzela

Owner: set to Paweł Horzela
Status: confirmedassigned

comment:3 Changed 7 years ago by Paweł Horzela

Status: assignedreview

comment:4 Changed 7 years ago by Paweł Horzela

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

Changed 7 years ago by Paweł Horzela

Attachment: 6236.patch added

Patch

comment:5 Changed 7 years ago by Wiktor Walc

Milestone: CKEditor 3.5.2

comment:6 Changed 7 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 7 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 7 years ago by Garry Yao

Attachment: 6236_2.patch added

comment:8 Changed 7 years ago by Garry Yao

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

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

Status: reviewreview_passed

comment:10 Changed 7 years ago by Garry Yao

Resolution: fixed
Status: review_passedclosed

Fixed with [6558].

Note: See TracTickets for help on using tickets.
© 2003 – 2017 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy