From cb1261cb98109a857f5d8db64893c98ac31cfee5 Mon Sep 17 00:00:00 2001 From: Dan Vanderkam Date: Fri, 23 Mar 2012 16:48:12 -0400 Subject: [PATCH] destroy mouseUpHandler --- dygraph.js | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/dygraph.js b/dygraph.js index 3f3b5a5..c07fe35 100644 --- a/dygraph.js +++ b/dygraph.js @@ -888,6 +888,7 @@ Dygraph.prototype.destroy = function() { // remove mouse event handlers Dygraph.removeEvent(this.mouseEventElement_, 'mouseout', this.mouseOutHandler); Dygraph.removeEvent(this.mouseEventElement_, 'mousemove', this.mouseMoveHandler); + Dygraph.removeEvent(this.mouseEventElement_, 'mousemove', this.mouseUpHandler_); removeRecursive(this.maindiv_); var nullOut = function(obj) { @@ -1182,7 +1183,7 @@ Dygraph.prototype.createDragInterface_ = function() { // If the user releases the mouse button during a drag, but not over the // canvas, then it doesn't count as a zooming action. - Dygraph.addEvent(document, 'mouseup', function(event) { + this.mouseUpHandler_ = function(event) { if (context.isZooming || context.isPanning) { context.isZooming = false; context.dragStartX = null; @@ -1198,7 +1199,9 @@ Dygraph.prototype.createDragInterface_ = function() { delete self.axes_[i].dragValueRange; } } - }); + }; + + Dygraph.addEvent(document, 'mouseup', this.mouseUpHandler_); }; /** -- 2.7.4