search

qooxdoo / qooxdoo

qooxdoo - Universal JavaScript Framework
http://qooxdoo.org
Other
766 stars 261 forks source link

keyboard events have target_check domnodes only. [object HTMLDocument] should be a target too (BZ#5700) #5770

Closed qx-bug-importer closed 8 years ago

qx-bug-importer commented 12 years ago

Gabriel Munteanu (jajalinux) wrote:

this bug is similar to http://bugzilla.qooxdoo.org/show_bug.cgi?id=4669 but for keyboard events.

compare the TARGET_CHECK static in both these files: http://qooxdoo.svn.sourceforge.net/viewvc/qooxdoo/trunk/qooxdoo/framework/source/class/qx/event/handler/Keyboard.js?revision=28889&view=markup and http://qooxdoo.svn.sourceforge.net/viewvc/qooxdoo/trunk/qooxdoo/framework/source/class/qx/event/handler/Mouse.js?revision=28889&view=markup

as you can see , for the mouse we have "qx.event.IEventHandler.TARGET_DOCUMENT + qx.event.IEventHandler.TARGET_WINDOW" in addition to what we have for keyboard events. adding them to Keyboard handler will remove this warning in applications: "There is no event handler for the event 'keydown' on target '[object HTMLDocument]'!"

assigned to Alexander Steitz (@asteitz)

qx-bug-importer commented 10 years ago

Martin Wittemann (@wittemann) wrote:

Move open issues to RESOLVED – LATER, whose last comment is older than a year.