Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prototype adapter: removeEvent doesn't remove all events #290

Closed
TorsteinHonsi opened this issue Jul 4, 2011 · 1 comment
Closed

Prototype adapter: removeEvent doesn't remove all events #290

TorsteinHonsi opened this issue Jul 4, 2011 · 1 comment

Comments

@TorsteinHonsi
Copy link
Collaborator

Steps to reproduce:

  1. Open http://jsfiddle.net/highcharts/8yhz3/
  2. Click "Destroy"
  3. Drag middle bar to resize the demo iframe
  4. Errors are thrown because the resize event handler is not properly removed.

To fix it, set up a unit test on adding and removing events on window, DOM objects as well as Highcharts object. Test this with element parameter, element + event name and element + event name + handler.

@eolsson
Copy link
Contributor

eolsson commented Jul 21, 2011

Events on 'window' where mistaken for custom events and the wrong stopObserving method was used.

@eolsson eolsson closed this as completed Jul 21, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants