Opened 10 years ago

Closed 10 years ago

#11574 closed Bug (fixed)

Contenteditable megabug with nested ul/li structure

Reported by: Aleksey Owned by: Piotrek Koszuliński
Priority: Normal Milestone: CKEditor 4.3.4
Component: Core : Editable Version:
Keywords: Cc:


1). Create simple html-document with the following simple structure:

<!DOCTYPE html>



<script src="ckeditor/ckeditor.js"></script>




<li>Section 1



<div contenteditable="true">Sub-section 1.1</div>

</li> <li>

<div contenteditable="true">Sub-section 1.2</div>







2). Open page in latest version of Firefox/Google Chrome (not tested others).

3). Click on "Sub-section 1.1" or "Sub-section 1.2" node. CKEditor will be loaded for the appropriate field.

4). Put the caret to the start of the edited node.

5). Press "Backspace" hotkey and you'll see that DOM-model is broken very hard:(((.

Attachments (1)

11574.html (674 bytes) - added by Piotrek Koszuliński 10 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 10 years ago by Piotrek Koszuliński

Keywords: ckeditor contenteditable bullets ul li bug removed
Milestone: CKEditor 4.3.4
Status: newconfirmed
Version: 4.3.2

Indeed, DOM is broken badly. CKEditor 4.3.3 is already closed, so we'll fix this in 4.3.4.

Changed 10 years ago by Piotrek Koszuliński

Attachment: 11574.html added

comment:2 Changed 10 years ago by Jakub Ś

Problem can be reproduced in all browsers from CKEditor 4.0. It doesn't occur in beta

On first Backspace press text is moved from lower li element to upper one.

On second Backspace press Blink, Webkit and IE act as if Back button (they go to previous page) was pressed while Firefox throws error: Message: TypeError: range is undefined
URI: /ckeditor4/core/editable.js
Line: 592
Code: path = range.startPath()

comment:3 Changed 10 years ago by Piotrek Koszuliński

Owner: set to Piotrek Koszuliński
Status: confirmedassigned

comment:4 Changed 10 years ago by Piotrek Koszuliński

Status: assignedreview

Pushed branch:t/11574 on dev and tests. It removes an odd line added in #9129.

comment:5 Changed 10 years ago by Frederico Caldeira Knabben

Status: reviewreview_passed

comment:6 Changed 10 years ago by Piotrek Koszuliński

Resolution: fixed
Status: review_passedclosed

Fixed on master with git:68a315d on dev and 465ecba on tests.

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