Fix how delta is set when deltaY and deltaX has a value

brandonaaron · brandonaaron · commit 599dcf3096e0 · 2013-11-14T11:57:19.000-05:00
diff --git a/jquery.mousewheel.js b/jquery.mousewheel.js
@@ -82,8 +82,8 @@
 
         // Firefox < 17 horizontal scrolling related to DOMMouseScroll event
         if ( 'axis' in orgEvent && orgEvent.axis === orgEvent.HORIZONTAL_AXIS ) {
+            deltaX = deltaY * -1;
             deltaY = 0;
-            deltaX = delta * -1;
         }
 
         // Set delta to be deltaY or deltaX if deltaY is 0 for backwards compatabilitiy
@@ -96,7 +96,7 @@
         }
         if ( 'deltaX' in orgEvent ) {
           deltaX = orgEvent.deltaX;
-          delta  = detlaX * -1;
+          if ( deltaY === 0 ) { delta  = deltaX * -1; }
         }
 
         // No change actually happened, no reason to go any further

Original file line number	Diff line number	Diff line change
`@@ -82,8 +82,8 @@`
`82`	`82`
`83`	`83`	`// Firefox < 17 horizontal scrolling related to DOMMouseScroll event`
`84`	`84`	`if ( 'axis' in orgEvent && orgEvent.axis === orgEvent.HORIZONTAL_AXIS ) {`
	`85`	`+ deltaX = deltaY * -1;`
`85`	`86`	`deltaY = 0;`
`86`		`- deltaX = delta * -1;`
`87`	`87`	`}`
`88`	`88`
`89`	`89`	`// Set delta to be deltaY or deltaX if deltaY is 0 for backwards compatabilitiy`
`@@ -96,7 +96,7 @@`
`96`	`96`	`}`
`97`	`97`	`if ( 'deltaX' in orgEvent ) {`
`98`	`98`	`deltaX = orgEvent.deltaX;`
`99`		`- delta = detlaX * -1;`
	`99`	`+ if ( deltaY === 0 ) { delta = deltaX * -1; }`
`100`	`100`	`}`
`101`	`101`
`102`	`102`	`// No change actually happened, no reason to go any further`