Community
Participate
Working Groups
Reproducible with Controls Demo -> TableViewer Tab in all browsers but Firefox. 1. Activate cell editors. 2. Click on a cell in the first column to activate the cell editor 3. Use TAB key to activate the CheckBox cell editor on the last column 4. Try to switch the checkbox selection with SPACE. In Firefox it's working fine. In Chrome, IE and Safari you have to press enter first in order to make SPACE selection works again.
Created attachment 205476 [details] Proposed patch This patch moves the _cancelEvent reset in the begining of SyncKeyEventUtil.js#intercept method to ensure clear state for every event. Remove browser switch (for IE) from KeyEventUtil.js#_isRelevantEvent.
Tim, please review the patch.
Comment on attachment 205476 [details] Proposed patch The ie-switch is something that pre-dates the EventHandler.js refactorings and should therefore not be necessary, but its unclear what exactly it was supposed to do in the first place. Therefore i'm unsure of this part. Everything else is fine.
(In reply to comment #3) > The ie-switch is something that pre-dates the EventHandler.js refactorings and > should therefore not be necessary, but its unclear what exactly it was supposed > to do in the first place. Therefore i'm unsure of this part. I also think that this ie-switch is not needed anymore after the EventHandler.js refactorings (I don't remember why it was introduced either). Applied patch to CVS HEAD and TTM branch.
