Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 16, 2025

Users were experiencing accidental edit/delete actions when scrolling vertically due to minor horizontal thumb movement triggering the swipe gesture.

Changes

Changed positionalThreshold in DeeprItemSwipable.kt from the default 56dp fixed distance to a dynamic 50% of item width:

// Before: Fixed 56dp threshold
positionalThreshold = SwipeToDismissBoxDefaults.positionalThreshold

// After: 50% of swipeable width
positionalThreshold = { totalDistance -> totalDistance * 0.5f }

On a typical 360dp screen, this increases the required swipe distance from ~56dp to ~180dp, making accidental triggers during vertical scrolling significantly less likely while keeping intentional swipes easily performable.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • dl.google.com
    • Triggering command: /usr/lib/jvm/temurin-17-jdk-amd64/bin/java --add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.lang.invoke=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED --add-exports=jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-opens=java.base/java.nio.charset=ALL-UNNAMED --add-opens=java.base/java.net=ALL-UNNAMED --add-opens=java.base/java.util.concurrent.atomic=ALL-UNNAMED --add-opens=java.xml/javax.xml.namespace=ALL-UNNAMED -Xmx2048m -Dfile.encoding=UTF-8 -Duser.country -Duser.language=en -Duser.variant -cp /home/REDACTED/.gradle/wrapper/dists/gradle-8.13-bin/5xuhj0ry160q40clulazy9h7d/gradle-8.13/lib/gradle-daemon-main-8.13.jar -javaagent:/home/REDACTED/.gradle/wrapper/dists/gradle-8.13-bin/5xuhj0ry160q40clulazy9h7d/gradle-8.13/lib/agents/gradle-instrumentation-agent-8.13.jar org.gradle.launcher.daemon.bootstrap.GradleDaemon 8.13 (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

This section details on the original issue you should resolve

<issue_title>Make that to swipe left or right it needs a 50% of horizontal swipe to trigger delete or edit</issue_title>
<issue_description>Because when I scroll sometimes I move a bit to left or right with the thumb and it triggers the swipe feature of edit or delete</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: yogeshpaliyal <9381846+yogeshpaliyal@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix swipe gesture threshold for delete or edit Increase swipe threshold to 50% to prevent accidental triggers during scroll Nov 16, 2025
Copilot AI requested a review from yogeshpaliyal November 16, 2025 16:28
Copilot finished work on behalf of yogeshpaliyal November 16, 2025 16:28
@github-actions
Copy link

📱 APK Build Complete!

Your debug APK has been built successfully and is ready for testing.

📥 Download APK

Download app-debug.apk

Note: Click the link above, scroll down to the "Artifacts" section, and download the app-debug artifact.

Retention: This artifact will be available for 3 days.

@github-actions
Copy link

🧪 Integrated Test Results

The integrated UI tests have completed.

View full test results: Test Run #19408558768

Test reports are available in the artifacts section of the workflow run.

@github-actions
Copy link

🧪 Integrated Test Results

The integrated UI tests have completed.

View full test results: Test Run #19408558701

Test reports are available in the artifacts section of the workflow run.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Make that to swipe left or right it needs a 50% of horizontal swipe to trigger delete or edit

2 participants