ANDROID-13422 Reset scroll offset on touch down (#12)

dpastor · web-flow · commit a8b05507663e · 2023-06-12T12:38:13.000+02:00
* ANDROID-13422 Reset scroll offset on touch down * ANDROID-13422 Improve comment
diff --git a/nestedscrollwebview/src/main/java/com/telefonica/nestedscrollwebview/NestedScrollWebView.java b/nestedscrollwebview/src/main/java/com/telefonica/nestedscrollwebview/NestedScrollWebView.java
@@ -208,6 +208,10 @@ public void onNestedTouchEvent(@NonNull MotionEvent ev) {
 
         if (actionMasked == MotionEvent.ACTION_DOWN) {
             mNestedYOffset = 0;
+            /* Not present on NestedScrollView code. We need to reset scroll offset on touch down,
+            as we found while testing that in some cases action was invoked with a non-zero scroll
+            offset, breaking page scroll. */
+            mScrollOffset[1] = 0;
         }
 
         MotionEvent vtev = MotionEvent.obtain(ev);

Original file line number	Diff line number	Diff line change
`@@ -208,6 +208,10 @@ public void onNestedTouchEvent(@NonNull MotionEvent ev) {`
`208`	`208`
`209`	`209`	`if (actionMasked == MotionEvent.ACTION_DOWN) {`
`210`	`210`	`mNestedYOffset = 0;`
	`211`	`+ /* Not present on NestedScrollView code. We need to reset scroll offset on touch down,`
	`212`	`+ as we found while testing that in some cases action was invoked with a non-zero scroll`
	`213`	`+ offset, breaking page scroll. */`
	`214`	`+ mScrollOffset[1] = 0;`
`211`	`215`	`}`
`212`	`216`
`213`	`217`	`MotionEvent vtev = MotionEvent.obtain(ev);`