robnyman / domassistant

Automatically exported from code.google.com/p/domassistant
1 stars 0 forks source link

DOMAssistant.Events.removeEvent and DOM Level 2 events #25

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. register listener using addEvent
2. unregister same listener using removeEvent

What is the expected output? What do you see instead?
removeEvent won't handle certain event names, specifically those with name 
prefix 'DOM' (e.g. 
mutation events). For browsers that support these events, try this:

$('#foo').addEvent('DOMNodeInserted', bar)
$('#foo').removeEvent('DOMNodeInserted', bar)// 'bar' still registered
//$('#foo').removeEventListener('DOMNodeInserted', bar, false)

Original issue reported on code.google.com by wormbo...@gmail.com on 27 Jan 2010 at 6:37

GoogleCodeExporter commented 9 years ago
A patch is ready for this fix. Could you help to test out the attached build?

Original comment by chengh...@gmail.com on 27 Jan 2010 at 7:29

Attachments:

GoogleCodeExporter commented 9 years ago
Previous fix breaks IE... use this one.

Original comment by chengh...@gmail.com on 27 Jan 2010 at 8:58

Attachments:

GoogleCodeExporter commented 9 years ago

Original comment by chengh...@gmail.com on 10 Mar 2010 at 9:10