Opened 9 years ago

Closed 9 years ago

#5828 closed Bug (fixed)

[ContentEditable] Infinite script loop can be easily fired

Reported by: Frederico Caldeira Knabben Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.4
Component: General Version:
Keywords: Confirmed Review? Cc:

Description

  1. Load the following HTML:
<p>Para 1</p>
<p><span contenteditable="false">Non Editable</span></p>
<p>Para 2</p>
  1. Hit CTRL+A to select all.
  1. Click the indentation button.

Current results:

The script will loop, blocking the browser. This happens when creating lists also.

Expected results:

All paragraphs (including the one with the read only content) should get indented.

This one is related to the contenteditable feature branch.

Change History (4)

comment:1 Changed 9 years ago by Frederico Caldeira Knabben

This happens even for inline operations, like applying styles.

comment:2 Changed 9 years ago by Garry Yao

Keywords: Confirmed added
Owner: set to Garry Yao
Status: newassigned

Infinite loop fixed with [5595], leave open for the unexpected result: Only first two paragraphs get indented.

comment:3 Changed 9 years ago by Garry Yao

Keywords: Review? added

Second issue is committed as [5599].

comment:4 Changed 9 years ago by Frederico Caldeira Knabben

Resolution: fixed
Status: assignedclosed

WFM now. Code review will be done all in a row on the branch.

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