In page key handlers are catching ENTER key on toolbar buttons

[Rajasimhan]

Ticket #4269 addressed the following issue - "Esc" and "Enter" keystrokes are not handled when a dialogue box is opened which causes the form post automatically.

But this issue is now re occurring in 3.5.2. Please check this and provide a fix.

Because of this issue whenever any plugins with a modal pop up window like Image Plugin, Links Plugin, Find & Replace are clicked either the window disappears immediately after it opens or the browser hangs as the JS goes in an infinite loop.

[Jakub Ś]

We were unable to reproduce the problem. Everything seems to be fine. Esc key behaves like Cancel button and Enter behaves like OK button. We couldn't also get any infinite loops or browser crashes.

Could you provide us with more details, like what OS are you using, which browsers does the issue concern? We would also be grateful for a detailed step by step scenario on how to reproduce the bug.

[Rajasimhan]

Hi,

Kindly bear with the delay. Please copy the attached file 'postissue.html' into the ckeditor\_samples folder.

1. Open the postissue.html. Editor will be rendered. Click inside the text area and press Alt-F10.
2. Tab till you reach 'Find and Replace' and press the enter key.
3. Form post will happen ( you would be taken to the 'sample_posteddata.php').

Browsers: This happens in FireFox and IE. OS: Windows. CKEditor Version: 3.5.2

[Rajasimhan]

Replying to Rajasimhan:

Hi,

Kindly bear with the delay. Please copy the attached file 'postissue.html' into the ckeditor\_samples folder.

1. Open the postissue.html. Editor will be rendered. Click inside the text area and press Alt-F10.
2. Tab till you reach 'Find and Replace' and press the enter key.
3. Form post will happen ( you would be taken to the 'sample_posteddata.php').

Browsers: This happens in FireFox and IE. OS: Windows. CKEditor Version: 3.5.2

​https://dev.ckeditor.com/attachment/ticket/7347/postissue.html

[Rajasimhan]

File to replicate the form post issue

[Jakub Ś]

Has happened from CKEditor version 3.2

[Frederico Caldeira Knabben]

Note that this issue is not related to #4269, which instead was addressing these keys on dialogs. This new ticket is talking about ENTER on the toolbar.

Please note that this issue is more related to an issue with your code than with the editor. The fact is that ENTER is happening on the page and your code is catching all instead of just the things you need to catch. We had already discussed this when talked about #4269 so it's not worth reopening this talk, but considering that this is an issue with your code and not CKEditor, it has lower priority.

[Rajasimhan]

Replication Files

[Frederico Caldeira Knabben]

This happens since [6303].

The dialog cover is created on keydown. At that point, the focus is set to it. So now the keyup event is caught by the dialog cover, not by the editor container.

This has nothing to do with CKEditor API features though.

[Frederico Caldeira Knabben]

It's enough to prevent key events on the cover at this point.

[Frederico Caldeira Knabben]

Fixed with [6676].