Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Android] Try to disambiguate taps from swipes #7448

Merged
merged 1 commit into from
Jan 15, 2025
Merged

Conversation

gaearon
Copy link
Collaborator

@gaearon gaearon commented Jan 14, 2025

Hackfix for callstack/react-native-pager-view#960.

Alternative to #7312 but doesn't require copy paste and also seems to work more reliably.

Comments explain how it works.

Will need some testing.

@arcalinea arcalinea temporarily deployed to capture-swipes - social-app PR #7448 January 14, 2025 03:41 — with Render Destroyed
Copy link

Old size New size Diff
6.86 MB 6.86 MB 0 B (0.00%)

Copy link
Member

@mozzius mozzius left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm fairly certain this is a solid fix. Tested on low-end android and it seems to eliminate the majority of rogue presses

@gaearon gaearon merged commit 96054f4 into main Jan 15, 2025
7 checks passed
gpp-0 added a commit to gpp-0/bsky-app-fork that referenced this pull request Jan 15, 2025
gaearon added a commit that referenced this pull request Jan 17, 2025
* [Android] Fix taps triggering while swiping

* Revert "[Android] Try to disambiguate taps from swipes (#7448)"

This reverts commit 96054f4.

* Update patch to match callstack/react-native-pager-view#961

* Make it symmetrical

---------

Co-authored-by: Dan Abramov <[email protected]>
gaearon pushed a commit that referenced this pull request Jan 20, 2025
gaearon added a commit that referenced this pull request Jan 20, 2025
* Bump pager

* Revert "[Android] Try to disambiguate taps from swipes (#7448)"

This reverts commit 96054f4.

---------

Co-authored-by: gpp-0 <[email protected]>
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.

3 participants