Ticket #6236 (closed Bug: fixed)

Opened 4 years ago

Last modified 3 years ago

[[IE]] Issues with Numbered list

Reported by: satya Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.5.3
Component: Core : Lists Version:
Keywords: IBM Cc: damo,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

Initial list.jpg (32.1 KB) - added by satya 4 years ago.
deletting list up to last item in 2nd level.jpg (42.5 KB) - added by satya 4 years ago.
list after pressing enter 3 times.jpg (31.4 KB) - added by satya 4 years ago.
6236.patch (675 bytes) - added by paho 4 years ago.
Patch
6236_2.patch (770 bytes) - added by garry.yao 4 years ago.

Change History

Changed 4 years ago by satya

Changed 4 years ago by satya

Changed 4 years ago by satya

comment:1 Changed 4 years ago by Saare

  • Status changed from new to confirmed
  • Component changed from General to Core : Lists

comment:2 Changed 4 years ago by paho

  • Status changed from confirmed to assigned
  • Owner set to paho

comment:3 Changed 4 years ago by paho

  • Status changed from assigned to review

comment:4 Changed 4 years ago by paho

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

Changed 4 years ago by paho

Patch

comment:5 Changed 4 years ago by wwalc

  • Milestone set to CKEditor 3.5.2

comment:6 Changed 4 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 4 years ago by garry.yao

  • Status changed from review to 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 4 years ago by garry.yao

comment:8 Changed 4 years ago by garry.yao

  • Status changed from review_failed to review
  • Owner changed from paho to garry.yao

comment:9 Changed 3 years ago by Saare

  • Status changed from review to review_passed

comment:10 Changed 3 years ago by garry.yao

  • Status changed from review_passed to closed
  • Resolution set to fixed

Fixed with [6558].

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