Opened 7 years ago

Closed 7 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 7 years ago.
5454_2.patch (1.6 KB) - added by brooks 7 years ago.

Download all attachments as: .zip

Change History (11)

comment:1 Changed 7 years ago by brooks

  • Keywords IE Firefox Chrome added

comment:2 Changed 7 years ago by garry.yao

  • Keywords IE Firefox Chrome removed

The bug only occur when the replacement text is empty.

comment:3 Changed 7 years ago by brooks

  • Owner set to brooks
  • Status changed from new to assigned

Changed 7 years ago by brooks

comment:4 Changed 7 years ago by brooks

  • Keywords Review? added; Confirmed removed

comment:5 Changed 7 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 7 years ago by brooks

comment:6 Changed 7 years ago by brooks

  • Keywords Review? added; Review- removed

comment:7 Changed 7 years ago by garry.yao

  • Keywords Review+ added; Review? removed

comment:8 Changed 7 years ago by brooks

  • Keywords confirmed added

fixed with 5345

comment:9 Changed 7 years ago by brooks

  • Resolution set to fixed
  • Status changed from assigned to closed
Note: See TracTickets for help on using tickets.
© 2003 – 2016 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy