Skip to content

Conversation

@IslamRustamov
Copy link

@IslamRustamov IslamRustamov commented Nov 14, 2025

Motivation

Application I'm working on has a bottom sheet with 3 snap points.

Recently I was informed that there is an issue with it:

  1. Bottom sheet is at the last snap point;
  2. User quickly swiped a little bit;
  3. Bottom sheet goes from last snap point to first one, ignoring the middle snap point.

So I was tasked to solve this by allowing user only one snap point change per swipe, but the library doesn't support this feature.

Here's the example of the problem:

Screen.Recording.2025-11-14.at.16.35.36.mov

We can see that most of the time, a little swipe is causing our bottom sheet to skip the middle snap point.

Here's the example of how it works with my changes:

Screen.Recording.2025-11-14.at.16.36.46.mov

We can see that now I can't skip the middle snap point and am forced to do one more swipe to get to the next snap point.

@IslamRustamov IslamRustamov force-pushed the feature/one-snap-point-per-swipe branch from d00d1a6 to 84ba93d Compare November 14, 2025 13:46
@IslamRustamov IslamRustamov marked this pull request as draft November 14, 2025 13:53
@IslamRustamov
Copy link
Author

IslamRustamov commented Nov 14, 2025

after I synced my fork - some problems jumped out, going to put in draft until I fix them

It stopped working the right way after these detents

the only problem that left is that animation is instant in case if I overdrag, will look into that but will be glad for help

upd: nervermind, this problem is existing in master, not the problem caused by my changes

Screen.Recording.2025-11-14.at.17.22.45.mov

@IslamRustamov IslamRustamov force-pushed the feature/one-snap-point-per-swipe branch from 84ba93d to 353a378 Compare November 14, 2025 14:15
@IslamRustamov IslamRustamov marked this pull request as ready for review November 14, 2025 15:00
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.

1 participant