Ticket #6159 (closed Bug: duplicate)

Opened 4 years ago

Last modified 3 years ago

When working with nested lists content may be accidentally deleted

Reported by: wwalc Owned by: paho
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

6159.patch (675 bytes) - added by paho 4 years ago.
Patch
6159_2.patch (2.0 KB) - added by garry.yao 4 years ago.

Change History

comment:1 Changed 4 years ago by wwalc

  • Status changed from new to confirmed

comment:2 Changed 4 years ago by wwalc

  • Component changed from General to Core : Lists

comment:3 Changed 4 years ago by paho

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

comment:4 Changed 4 years ago by paho

  • Status changed from assigned to review

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 4 years ago by paho

Patch

comment:5 Changed 4 years ago by paho

  • Keywords IE added

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

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

comment:8 Changed 4 years ago by paho

  • Status changed from review_failed to review

comment:9 Changed 3 years ago by wwalc

  • Milestone set to CKEditor 3.5.2

comment:10 Changed 3 years ago by garry.yao

  • Status changed from review to closed
  • Resolution set to duplicate

DUP of #6236.

comment:11 Changed 3 years ago by wwalc

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