add option to disable the animation in scrollToIndex when gestureRele…#36
add option to disable the animation in scrollToIndex when gestureRele…#36gvlekke wants to merge 1 commit into
Conversation
|
@gvlekke Well done. I was trying to find a warkaround for that very issue |
|
@msevestre did you test it yourself already? In my case the experience approved a lot. What is your experience? |
|
I would love to test this out as well! I'll check it out this weekend! |
|
@gvlekke Haven't tried it yet but will hopefully by the end of today or tomorrow |
|
@gvlekke I just tried it on a couple devices. Felt much better on an older device and maybe slightly faster on brand new Pixel. I'd say it does not hurt to have the option :) |
|
@gvlekke I should add that with the animation turned off, the behavior on my ios simulator does not look as good (e.g. transition from one slide to the other is way too fast). But looks better on devices... |
|
@msevestre yeah that is also what i noticed. Btw we are using now a self made slider that is way faster. It just uses a flatlist. Only problem is that we have some white space when you go from the first item to the last item... When I have time im also going to check if can make some library of that. Then we can compare our stuff :) |
What did you do:
I noticed that the slider feels a little bit slow when the gestureRelease triggers.
It was because of the animation in the scrollToIndex. So I added a option to disable that if you dont want the animation.
Maybe a good idea?
Dont know if it is a good idea to have this as a default behaviour? Then the carousel is fast for everyone.
Does this relate to any issue(s)? If so which one(s)?
nope
Checklist:
yarn lintandyarn flowstill in yarn flow things break and dont know how to fix that? can you explain that?