Ticket #5064 (closed Bug: invalid)

Opened 5 years ago

Last modified 4 years ago

Enter button is not handled properly while activating toolbar controls.

Reported by: pranav Owned by:
Priority: Normal Milestone:
Component: UI : Toolbar Version:
Keywords: Oracle Cc: Senthil, Pranav


Enter is not captured for the toolbar buttons which causes the form post automatically. To reproduce the issue, follow the following steps:

  1. Press Alt+F10 to activate toolbar control. Varify that source button is selected.
  1. Press Enter button to switch to source mode.

You will observe that form post is happening. Same thing is happening for the "Maximize" button too.

Sample file to reproduce the issue is attached with this ticket.


replacebycode_enter.html (2.9 KB) - added by pranav 5 years ago.
Sample file to reproduce the issue
replacebycode_enter.2.html (2.6 KB) - added by alfonsoml 5 years ago.
Adjusted test file

Change History

Changed 5 years ago by pranav

Sample file to reproduce the issue

comment:1 Changed 5 years ago by fredck

  • Priority changed from High to Normal
  • Milestone set to CKEditor 3.x

For now, your custom key handling system should check whether the key is being pressed inside elements that should not cause form posting.

comment:2 Changed 5 years ago by garry.yao

  • Status changed from new to closed
  • Version 3.1 deleted
  • Resolution set to invalid

Again, others might find the event bubbling useful, we can't just kill it only because of the way you choose to implement application.

comment:3 Changed 5 years ago by Senthil

If the user is using keyboard keys to access the toolbar control, he would not be able to access the control at all. Becuase the form post is happening while pressing the enter key. Since it is an accessibility related(critical to us), we need a patch for this.

Changed 5 years ago by alfonsoml

Adjusted test file

comment:4 Changed 5 years ago by alfonsoml

  • Milestone CKEditor 3.x deleted

The file shows how to detect if the event is directed at the form elements or to something else in the page and avoid submitting the form in that case.

comment:5 Changed 5 years ago by Senthil

  • Priority changed from Normal to High
  • Status changed from closed to reopened
  • Resolution invalid deleted

This is creating unnecessary form post in our pages and our page functionality is controlled by the different framework and that cannot be changed for this new behavior. So this issue needs to be addressed ASAP since it is a critical issue.

comment:6 Changed 4 years ago by Senthil

Please close this ticket since the workaround(received from Garry) fixes the reported problem.

comment:7 Changed 4 years ago by fredck

  • Priority changed from High to Normal
  • Status changed from reopened to closed
  • Resolution set to invalid
Note: See TracTickets for help on using tickets.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy