We are migrating CKEditor issue tracking to GitHub. Please, use GitHub to report any new issues.

The former tracking system (this website) will still be available in the read-only mode. All issues reported in the past will still be available publicly and can be referenced.

Important: we decided not to transfer all the tickets to GitHub, as many of them are not reproducible anymore or simply no longer requested by the community.
If the issue you are interested in, can be still reproduced in the latest version of CKEditor, feel free to report it again on GitHub.
At the same time please note that issues reported on this website are still taken into consideration when picking up candidates for next milestones.

To report a new issue, go to https://github.com/ckeditor/ckeditor-dev/issues and follow the instructions in the issue template.

Context Navigation

← Previous Change
Ticket History
Next Change →

Changes between Initial Version and Version 2 of Ticket #12332

Timestamp:: Sep 8, 2014, 8:03:30 AM (10 years ago)
Author:: Piotrek Koszuliński
Comment:

Legend:

: Unmodified
: Added
: Removed
: Modified

Ticket #12332
- Property Status changed from new to confirmed

Ticket #12332 – Description

-                      initial
+                      v2
 This is a follow up of #12327.
 Current implementation is really fragile to canceling events. Eg. a developer might cancel keydown event for whatever reason, and undo will not perform preparation code.
+Current implementation is really fragile to code execution order. If undo manager is initialised before some code that cancels keydown undo manager's listener will be executed, but if the order is opposite, then it won't.
 Our goal priority should be below default {{{10}}}.
+The listeners should be added with low priorities (999), so they are always handled at the end - if nothing else handled them.