Merge pull request #2072 from IgniteUI/skrastev/fix-267598-19.2

ChronosSF · web-flow · commit 7bc51356d6e1 · 2020-06-19T14:14:11.000+03:00
Move thin scrollbars to show only after scrolling has started and not on touchstart.
diff --git a/src/js/modules/infragistics.ui.scroll.js b/src/js/modules/infragistics.ui.scroll.js
@@ -2421,8 +2421,6 @@
 			this._offsetDirection = 0;
 
 			this._igScollTouchPrevented = false;
-
-			this._showScrollbars(true);
 		},
 
 		_onTouchMoveContainer: function (event) {
@@ -2510,11 +2508,12 @@
 			}
 
 			if (scrolledXY.x === 0 && scrolledXY.y === 0) {
-			    this._igScollTouchPrevented = true;
+				this._igScollTouchPrevented = true;
 			}
 
 			//On Safari preventing the touchmove would prevent default page scroll behaviour even if there is the element doesn't have overflow
 			if (!this._igScollTouchPrevented && event.cancelable) {
+				this._showScrollbars(true);
 				event.preventDefault();
 			}
 		},

Original file line number	Diff line number	Diff line change
`@@ -2421,8 +2421,6 @@`
`2421`	`2421`	`this._offsetDirection = 0;`
`2422`	`2422`
`2423`	`2423`	`this._igScollTouchPrevented = false;`
`2424`		`-`
`2425`		`- this._showScrollbars(true);`
`2426`	`2424`	`},`
`2427`	`2425`
`2428`	`2426`	`_onTouchMoveContainer: function (event) {`
`@@ -2510,11 +2508,12 @@`
`2510`	`2508`	`}`
`2511`	`2509`
`2512`	`2510`	`if (scrolledXY.x === 0 && scrolledXY.y === 0) {`
`2513`		`- this._igScollTouchPrevented = true;`
	`2511`	`+ this._igScollTouchPrevented = true;`
`2514`	`2512`	`}`
`2515`	`2513`
`2516`	`2514`	`//On Safari preventing the touchmove would prevent default page scroll behaviour even if there is the element doesn't have overflow`
`2517`	`2515`	`if (!this._igScollTouchPrevented && event.cancelable) {`
	`2516`	`+ this._showScrollbars(true);`
`2518`	`2517`	`event.preventDefault();`
`2519`	`2518`	`}`
`2520`	`2519`	`},`