Opened 13 years ago

Closed 13 years ago

Last modified 12 years ago

#1405 closed defect (fixed)

dojo.event.browser.* toLowerCase() causes problems with case sensative event names

Reported by: chris@… Owned by: alex
Priority: high Milestone:
Component: Events Version: 0.2
Keywords: addListener DOMMouseScroll toLowerCase Cc:
Blocked By: Blocking:

Description

In both dojo.event.browser.addListener() and dojo.event.browser.removeListener(), the "evtName" is forced to lower case. This causes problems when adding/removing listeners to Mozilla's "DOMMouseScroll" event. Commenting the toLowerCase() lines works, but Alex Russell suggested white listing all case sensative event handler names.

According to XULPlanet, the following event names should be white listed:

CheckboxStateChange
DOMAttrModified
DOMMenuItemActive
DOMMenuItemInactive
DOMMouseScroll
DOMNodeInserted
DOMNodeRemoved
RadioStateChange

dojo.event.connect may also be affected, but I haven't investigated.

Current workaround is to modify the dojo source and comment out the offending lines.

Change History (4)

comment:1 Changed 13 years ago by dylan

severity: majorblocker

comment:2 Changed 13 years ago by dylan

Resolution: fixed
Status: newclosed

(In [6204]) dojo.event.browser.* toLowerCase() causes problems with case sensative event names, fixes #1405

comment:3 Changed 13 years ago by dylan

Resolution: fixed

(In [6211]) dojo.event.browser.* toLowerCase() causes problems with case sensative event names, fixes #1405, take 2

comment:4 Changed 12 years ago by (none)

Milestone: 0.4

Milestone 0.4 deleted

Note: See TracTickets for help on using tickets.