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

fix: Weird snap behavior (Android) #10

Closed
wants to merge 1 commit into from
Closed

fix: Weird snap behavior (Android) #10

wants to merge 1 commit into from

Conversation

djorkaeffalexandre
Copy link
Contributor

@djorkaeffalexandre djorkaeffalexandre commented May 29, 2020

@rgommezz

Motivation

If you refer to my Snack repo here, you can see that have some weird behaviors at Android, sometimes when you try to go to an intermediate snap, it open at the latest snap point but all the values are at the right point.
I just rollback some commits searching for some code that's working and found this piece of code that fixes the issue, so, I just put it back.
If you have a better solution just let me know.
Thanks in advance.

@betoharres
Copy link

Thanks for the quick fix, I'm using your branch on my project! This PR fixes #16 I've opened
🇧🇷

@rgommezz
Copy link
Owner

Closing as discussed on #7

@rgommezz rgommezz closed this Jun 20, 2020
@djorkaeffalexandre
Copy link
Contributor Author

djorkaeffalexandre commented Jun 20, 2020

@rgommezz I've made a new Snack that can explain my point:
https://snack.expo.io/@djorkaeff/bottom-sheet-snap
Just test this and see the code, it should open at the medium point, but this always open at the top level and the gestures are weird at android devices, if you try to run this with my changes here it'll work as expected..

@FRizzonelli
Copy link

@rgommezz I can confirm that I've the same issue and PR from @djorkaeffalexandre works like a charm. Can you maybe merge it?

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.

4 participants