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

Changes between Version 3 and Version 4 of Ticket #12324, comment 4

Timestamp:: Aug 22, 2014, 11:09:55 AM (11 years ago)
Author:: Artur Delura
Comment:

Legend:

: Unmodified
: Added
: Removed
: Modified

Ticket #12324, comment 4

-                      v3
+                      v4
 {{{<p>bar ^baz</p>}}}
-Please confirm for now.
 Expected behaviour is that when we change selection after making changes then snapshot should be saved. But it does not save snapshot when we put caret at the beginning of node (see point 6 - here we put caret at the same beginning of paragraph).
+Deeper investigation brings me here:
+'''Deeper investigation brings me here:'''
 Saving snapshot is handled [https://github.com/ckeditor/ckeditor-dev/blob/master/plugins/undo/plugin.js#L973-L978 here] But it's unreliable, because editable has margin on each side, so clicking in margin area does not fire click event, but change selection.
+'''Proposed solution:'''
+To cover 100% use case let's listen {{{selectionChange}}} event and in callback call {{{onNavigationKey}}} method.
+{{{
+this.undoManager.editor.on( 'selectionChange', function() {
+    that.onNavigationKey();
+} );
+}}}