Events: Unbind scroll events properly

Closes jquery-archive#8451 Fixes jquery-archive#8250
arschmitz · Jul 4, 2016 · 76e9a47 · 76e9a47
1 parent 0756d6c
commit 76e9a47
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/js/events/scroll.js b/js/events/scroll.js
@@ -59,8 +59,7 @@ $.event.special.scrollstart = {
 			event.type = originalEventType;
 		}
 
-		// iPhone triggers scroll after a small delay; use touchmove instead
-		$this.bind( scrollEvent, function( event ) {
+		var scrollStartHandler = $.event.special.scrollstart.handler = function ( event ) {
 
 			if ( !$.event.special.scrollstart.enabled ) {
 				return;
@@ -74,10 +73,13 @@ $.event.special.scrollstart = {
 			timer = setTimeout( function() {
 				trigger( event, false );
 			}, 50 );
-		} );
+		};
+
+		// iPhone triggers scroll after a small delay; use touchmove instead
+		$this.on( scrollEvent, scrollStartHandler );
 	},
 	teardown: function() {
-		$( this ).unbind( scrollEvent );
+		$( this ).off( scrollEvent, $.event.special.scrollstart.handler );
 	}
 };