Skip to content

Alternatives to msrv_speed_ms_per_inch #24

@mvgreen

Description

@mvgreen

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:

  1. ms/inch is not a speed, it's inversion of speed, so the word "speed" in the attribute name is very confusing;
  2. 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.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions