Opened 8 years ago

Closed 8 years ago

#5454 closed Bug (fixed)

After replaced success, the popup window couldn't been closed and a js error occured

Reported by: brooks Owned by: brooks
Priority: Normal Milestone: CKEditor 3.3
Component: General Version: SVN (CKEditor) - OLD
Keywords: confirmed Review+ Cc:

Description

Load the replace by class sample page; 1.Open 'Replace' dialog and fill in 'are'; 2.click the 'Replace' buttton 2 times; 3.click the 'Cancel' bbutton or the 'x' button to close the popup window

Expected Result: the popup window is closed. Actual Result: a js error throws and you couldn't close the popup window any more

Attachments (2)

5454.patch (731 bytes) - added by brooks 8 years ago.
5454_2.patch (1.6 KB) - added by brooks 8 years ago.

Download all attachments as: .zip

Change History (11)

comment:1 Changed 8 years ago by brooks

Keywords: IE Firefox Chrome added

comment:2 Changed 8 years ago by Garry Yao

Keywords: IE Firefox Chrome removed

The bug only occur when the replacement text is empty.

comment:3 Changed 8 years ago by brooks

Owner: set to brooks
Status: newassigned

Changed 8 years ago by brooks

Attachment: 5454.patch added

comment:4 Changed 8 years ago by brooks

Keywords: Review? added; Confirmed removed

comment:5 Changed 8 years ago by Garry Yao

Keywords: Review- added; Review? removed

The patch is a workaround instead of a fix, the collapsed selection (cursor) is not visible after the dialog is closed.

Changed 8 years ago by brooks

Attachment: 5454_2.patch added

comment:6 Changed 8 years ago by brooks

Keywords: Review? added; Review- removed

comment:7 Changed 8 years ago by Garry Yao

Keywords: Review+ added; Review? removed

comment:8 Changed 8 years ago by brooks

Keywords: confirmed added

fixed with 5345

comment:9 Changed 8 years ago by brooks

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