Skip to content

Conversation

troZee
Copy link
Collaborator

@troZee troZee commented Dec 9, 2019

Summary

On Android when ViewPager is scrolling with swipe gesture all content(including all Touchables) are disabled. On iOS it's not - all touchables are interactable while swiping so when viewpager content has button or other clickable components they are clicked.

Close #73

Test Plan

Click on button, while scrolling. Check if no of likes increased.

Compatibility

OS Implemented
iOS
Android

Checklist

  • I have tested this on a device and a simulator
  • I updated the typed files (TS and Flow)

@troZee troZee added the ios label Dec 9, 2019
@troZee troZee requested review from ferrannp and dratwas December 9, 2019 10:06
Copy link

@oliverdolgener oliverdolgener left a comment

Choose a reason for hiding this comment

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

LGTM

@troZee troZee force-pushed the @troZee/disable-touch-events branch from 3ee060c to 50cd78d Compare December 10, 2019 20:12
@troZee troZee requested a review from dratwas December 10, 2019 20:12
@dratwas
Copy link

dratwas commented Dec 11, 2019

Approved, however, I don't like the any type :)

@KingAmo
Copy link

KingAmo commented Dec 11, 2019

may i ask when will this PR be merged and on npm?

@ferrannp ferrannp merged commit 0f1cdfe into master Dec 11, 2019
@ferrannp ferrannp deleted the @troZee/disable-touch-events branch December 11, 2019 12:21
grahammendick added a commit to grahammendick/navigation that referenced this pull request Jul 23, 2020
Used to do this by intercepting touch but there's no ios equivalent. Took the idea from callstack/react-native-pager-view#124
alpha0010 added a commit to alpha0010/react-native-viewpager that referenced this pull request Mar 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[V2] Disable content touches when ViewPager is scrolling on iOS
5 participants