Opened 11 years ago
Closed 11 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: |
Description
1). Create simple html-document with the following simple structure:
<!DOCTYPE html>
<html>
<head>
<script src="ckeditor/ckeditor.js"></script>
</head>
<body>
<ul>
<li>Section 1
<ul>
<li>
<div contenteditable="true">Sub-section 1.1</div>
</li> <li>
<div contenteditable="true">Sub-section 1.2</div>
</li>
</ul>
</li>
</ul>
</body>
</html>
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)
Change History (7)
comment:1 Changed 11 years ago by
| Keywords: | ckeditor contenteditable bullets ul li bug removed |
|---|---|
| Milestone: | → CKEditor 4.3.4 |
| Status: | new → confirmed |
| Version: | 4.3.2 |
Changed 11 years ago by
| Attachment: | 11574.html added |
|---|
comment:2 Changed 11 years ago by
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 11 years ago by
| Owner: | set to Piotrek Koszuliński |
|---|---|
| Status: | confirmed → assigned |
comment:4 Changed 11 years ago by
| Status: | assigned → review |
|---|
Pushed branch:t/11574 on dev and tests. It removes an odd line added in #9129.
comment:5 Changed 11 years ago by
| Status: | review → review_passed |
|---|
comment:6 Changed 11 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
Fixed on master with git:68a315d on dev and 465ecba on tests.
Indeed, DOM is broken badly. CKEditor 4.3.3 is already closed, so we'll fix this in 4.3.4.