Opened 7 years ago

Closed 6 years ago

Last modified 6 years ago

#6159 closed Bug (duplicate)

When working with nested lists content may be accidentally deleted

Reported by: Wiktor Walc Owned by: Paweł Horzela
Priority: Normal Milestone:
Component: Core : Lists Version: 3.2.2
Keywords: IE Cc:

Description

In a certain situation, list items disappear.

Steps to reproduce

  • Click bullet icon in editor
  • Type 'test1' , press enter
  • Indent the bullet
  • Type 'test11' , press enter
  • Type 'test12' , press enter
  • Type 'test13'
  • Move cursor behind test12
  • Press enter 3 times
  • Press backspace 4 times ( deleting the bullets ) . Note that test13 ends up one level lower
  • Place cursor after 'test11'
  • Press enter 3 times

After doing this, you will notice that all the subbullets (test11 and test12 ) are gone.

Tested on IE6, using http://ckeditor.com/demo

Also tested in IE8, the behaviour is a little different, but the result is the same (list items are gone).

It looks like this issue was introduced with [5380] (#4476).

Attachments (2)

6159.patch (675 bytes) - added by Paweł Horzela 7 years ago.
Patch
6159_2.patch (2.0 KB) - added by Garry Yao 7 years ago.

Download all attachments as: .zip

Change History (13)

comment:1 Changed 7 years ago by Wiktor Walc

Status: newconfirmed

comment:2 Changed 7 years ago by Wiktor Walc

Component: GeneralCore : Lists

comment:3 Changed 7 years ago by Paweł Horzela

Owner: set to Paweł Horzela
Status: confirmedassigned

comment:4 Changed 7 years ago by Paweł Horzela

Status: assignedreview

I'm sorry for attaching two patches - they are the same, I've just clicked 'Atach file' button two times by mistake.

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

Changed 7 years ago by Paweł Horzela

Attachment: 6159.patch added

Patch

comment:5 Changed 7 years ago by Paweł Horzela

Keywords: IE added

comment:6 Changed 7 years ago by Garry Yao

A much reduced procedural:

  1. Load the following content and selection;
    <ol>
        <li>
            test1
            <ol>
                <li>
                    test11^</li>
            </ol>
        </li>
    </ol>
    
    
  1. Press one 'Enter' and 3 'Backspace' which should remain the list structure;
  2. Now press 3 'Enter' and notice the result of the last one;
    • Actual Result: Last enter removes also list item 'test11'.

comment:7 Changed 7 years ago by Garry Yao

Status: reviewreview_failed

The bug is not related to [5380] but the 'Backspace' key that handled by IE (not us) that transform the good list into a malformed one:

<ol>
    <li>
        test1
    </li>
    <ol>
         <li>
             test11^</li>
    </ol>
</ol>

We should show some tolerance to it when transforming.

Changed 7 years ago by Garry Yao

Attachment: 6159_2.patch added

comment:8 Changed 7 years ago by Paweł Horzela

Status: review_failedreview

comment:9 Changed 7 years ago by Wiktor Walc

Milestone: CKEditor 3.5.2

comment:10 Changed 6 years ago by Garry Yao

Resolution: duplicate
Status: reviewclosed

DUP of #6236.

comment:11 Changed 6 years ago by Wiktor Walc

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