Opened 7 years ago

Closed 6 years ago

Last modified 6 years ago

#5546 closed Bug (fixed)

SCAYT breaks undo system

Reported by: garry.yao Owned by:
Priority: Normal Milestone: CKEditor 3.3
Component: UI : Spell Checker Version: 3.0
Keywords: Confirmed Cc:


Reproducing Procedures

  1. Open 'replacebyclass' sample page;
  2. Enable 'SCAYT' and wait for SCAYT to be activiated;
  3. Put cursor inside the word 'sample' as following:
  4. Open 'Smiley' dialog to insert one default icon at the cursor position;
  5. Wait until the SCAYT has finished marking the mis-spelled word 'sam' and 'ple' respectively;
  6. Press 'Undo' button to revert the smiley insertion;
    • Actual Result: The SCAYT mark line was reverted but not the smiley icon.
    • Expected Result: Both smiley icon insertion and SCAYT mark get reverted.

Attachments (2)

5546.patch (2.5 KB) - added by 6 years ago.
5546_2.patch (4.1 KB) - added by 6 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 7 years ago by garry.yao

Note that there're two problems involved here:

  1. First of all, SCAYT should refresh after undo;
  2. Second, SCAYT has to notify our undo system of the newly added SCAYT word markers.

Changed 6 years ago by

Changed 6 years ago by

comment:2 Changed 6 years ago by

SCAYT instance status check was added ( was led to error whet text source edit )

comment:3 Changed 6 years ago by garry.yao

  • Resolution set to fixed
  • Status changed from new to closed

Fixed with [5485] with dead code blocks removed.

comment:4 Changed 6 years ago by alfonsoml

This has broken the undo system if there is no instance named "editor1" #5685

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