Hello and thank you for your library, it helped me a lot! However, I found an issue in your implementation.
In fact there are two issues in one:
- ms/inch is not a speed, it's inversion of speed, so the word "speed" in the attribute name is very confusing;
- It would be better to have some alternative attribute to define constant snap time.
I'll explain the second issue as it is more important.
If you swipe your recyclerview on a small screen in portrait mode with (for example) msrv_speed_ms_per_inch=50, it will change the page pretty fast. But if you'll try it on a bigger device (on a tablet), it will scroll really slow, because it has a larger surface and it takes more time to reach target point.
Instead, it would be better to declare a constant time for snap helper, that will be equally fast on both devices.
Hello and thank you for your library, it helped me a lot! However, I found an issue in your implementation.
In fact there are two issues in one:
I'll explain the second issue as it is more important.
If you swipe your recyclerview on a small screen in portrait mode with (for example) msrv_speed_ms_per_inch=50, it will change the page pretty fast. But if you'll try it on a bigger device (on a tablet), it will scroll really slow, because it has a larger surface and it takes more time to reach target point.
Instead, it would be better to declare a constant time for snap helper, that will be equally fast on both devices.