issue-10971-swipe-down-gesture-too-sensitive

app/build.gradle

@@ -199,8 +199,7 @@ dependencies {
     // name and the commit hash with the commit hash of the (pushed) commit you want to test
     // This works thanks to JitPack: https://jitpack.io/
     implementation 'com.github.TeamNewPipe:nanojson:1d9e1aea9049fc9f85e68b43ba39fe7be1c1f751'
-    // WORKAROUND: v0.24.2 can't be resolved by jitpack -> use git commit hash instead
-    implementation 'com.github.TeamNewPipe:NewPipeExtractor:176da72cb4c3ec4679211339b0e59f6b01bf2f52'
+    implementation 'com.github.TeamNewPipe:NewPipeExtractor:v0.24.2'
     implementation 'com.github.TeamNewPipe:NoNonsense-FilePicker:5.0.0'
 
 /** Checkstyle **/

app/src/main/java/org/schabi/newpipe/fragments/detail/VideoDetailFragment.java

@@ -2308,7 +2308,6 @@ private void setupBottomPlayer() {
             @Override
             public void onStateChanged(@NonNull final View bottomSheet, final int newState) {
                 updateBottomSheetState(newState);
-
                 switch (newState) {
                     case BottomSheetBehavior.STATE_HIDDEN:
                         moveFocusToMainFragment(true);
@@ -2369,6 +2368,26 @@ && isPlayerAvailable()
                 }
             }
 
+            private String getStateName(final int state) {
+                switch (state) {
+                    case BottomSheetBehavior.STATE_DRAGGING:
+                        return "DRAGGING";
+                    case BottomSheetBehavior.STATE_SETTLING:
+                        return "SETTLING";
+                    case BottomSheetBehavior.STATE_EXPANDED:
+                        return "EXPANDED";
+                    case BottomSheetBehavior.STATE_COLLAPSED:
+                        return "COLLAPSED";
+                    case BottomSheetBehavior.STATE_HIDDEN:
+                        return "HIDDEN";
+                    case BottomSheetBehavior.STATE_HALF_EXPANDED:
+                        return "HALF_EXPANDED";
+                    default:
+                        return "UNKNOWN";
+                }
+            }
+
+
             @Override
             public void onSlide(@NonNull final View bottomSheet, final float slideOffset) {
                 setOverlayLook(binding.appBarLayout, behavior, slideOffset);

app/src/main/java/org/schabi/newpipe/player/gesture/CustomBottomSheetBehavior.java

@@ -3,6 +3,7 @@
 import android.content.Context;
 import android.graphics.Rect;
 import android.util.AttributeSet;
+import android.util.DisplayMetrics;
 import android.view.MotionEvent;
 import android.view.View;
 import android.widget.FrameLayout;
@@ -22,6 +23,9 @@ public class CustomBottomSheetBehavior extends BottomSheetBehavior<FrameLayout>
     public CustomBottomSheetBehavior(@NonNull final Context context,
                                      @Nullable final AttributeSet attrs) {
         super(context, attrs);
+        final DisplayMetrics displayMetrics = context.getResources().getDisplayMetrics();
+        final int screenHeight = displayMetrics.heightPixels; //record the screen height
+        dragThreshold = (float) (0.05 * screenHeight); // Adjust this value as needed
     }
 
     Rect globalRect = new Rect();
@@ -30,6 +34,10 @@ public CustomBottomSheetBehavior(@NonNull final Context context,
             R.id.detail_content_root_layout, R.id.relatedItemsLayout,
             R.id.itemsListPanel, R.id.view_pager, R.id.tab_layout, R.id.bottomControls,
             R.id.playPauseButton, R.id.playPreviousButton, R.id.playNextButton);
+    private float startY = 0f;
+    private float totalDrag = 0f;
+    private final float dragThreshold;
+
 
     @Override
     public boolean onInterceptTouchEvent(@NonNull final CoordinatorLayout parent,
@@ -39,6 +47,9 @@ public boolean onInterceptTouchEvent(@NonNull final CoordinatorLayout parent,
         if (event.getAction() == MotionEvent.ACTION_CANCEL
                 || event.getAction() == MotionEvent.ACTION_UP) {
             skippingInterception = false;
+            //reset the variables
+            totalDrag = 0f;
+            startY = 0f;
         }
 
         // Found that user still swiping, continue following
@@ -77,6 +88,28 @@ public boolean onInterceptTouchEvent(@NonNull final CoordinatorLayout parent,
                 }
             }
         }
+        // Implement the distance threshold logic, when the gesture move distance
+        // is shorter than the threshold, just do not intercept the touch event.
+        switch (event.getActionMasked()) {
+            case MotionEvent.ACTION_DOWN://initialize the tracked total distance
+                startY = event.getY();
+                totalDrag = 0f;
+                break;
+            case MotionEvent.ACTION_MOVE://accumulate the total distance
+                final float currentY = event.getY();
+                totalDrag += Math.abs(currentY - startY);
+                startY = currentY;
+                if (totalDrag < dragThreshold) {
+                    return false; // Do not intercept touch events yet if shorter than threshold
+                } else {
+                    return super.onInterceptTouchEvent(parent, child, event);
+                }
+            case MotionEvent.ACTION_UP:
+            case MotionEvent.ACTION_CANCEL://reset the drag distance and start point
+                totalDrag = 0f;
+                startY = 0f;
+                break;
+        }
 
         return super.onInterceptTouchEvent(parent, child, event);
     }