[patch] [cla] ESC keyboard event not working on webkit

[bill]

See enterKeyPress.html. ESCAPE key does not produce any event.

This is based on #9506 but just filing a separate ticket for clarity, since it's a general events problem, not something wrong w/dijit.

[bill]

(In [18744]) Automated test for keyboard event handling, showing safari 4 problems with ESC and ctrl-q. Refs #9511, #9527, #9528.

Maybe this should be added to runTests.html although we don't have any robot tests in there currently.

[Adam Peller]

Safari never had keyboard support. consider for 1.4

[bill]

Note that dijit's _FormWidget.js has some workaround code related to this, in _onKeyDown:

}else if(dojo.isWebKit){ // ESCAPE needs help making it into keypress
        te = document.createEvent('Events');
        te.initEvent('keypress', true, true);
        te.keyCode = dojo.keys.ESCAPE;
        te.shiftKey = e.shiftKey;
        e.target.dispatchEvent(te);
}

Obviously it would be better to handle that in core.

[Douglas Hays]

Patch to send fake keypress on ESC

[Douglas Hays]

This patch seems to fix #6184 as well and will allow redundant fake keypress code to be removed from _FormWidget.s and popup.js in dijit.

[bill]

Cool, this looks great. ESC is unprintable, I don't know why it wasn't categorized that way to begin with. Probably just an oversight?

I'm marking this as high-priority to remember to add it before we go beta.

[Becky Gibson]

this patch also fixes #9928

[Douglas Hays]

(In [20306]) Fixes #6184, #9528, #6184 !strict. Forced Esc key to generate fake keypress on WebKit?. Removed keydown special WebKit/Esc? processing from dijit's popup.js and _FormWidget.js. Verified that dijit's robot tests ComboBox_a11y.html, Dialog_a11y.html, and Menu_a11y.html now run without errors on Safari4/Windows.