woltapp / wolt_modal_sheet

This package provides a responsive modal with multiple pages, motion animation for page transitions, and scrollable content within each page.
MIT License
505 stars 64 forks source link

Enable Dismissal of Sliver Modal Sheet on Downward Drag at Scroll Edge #331

Open sheeroo opened 1 month ago

sheeroo commented 1 month ago

when using a sliver modal sheet with scrollable content like a list, dismissing the modal by dragging it down isn't possible once the scrollable content has reached its top edge (meaning it can't scroll up any further).

This behavior differs from common interaction patterns in other apps, where a downward drag at the top of a scrollable list dismisses the modal.

Adding this feature would enhance the modal's usability by allowing it to be dismissed with a downward swipe when the scroll position is at the top edge.

ulusoyca commented 1 month ago

Hi @sheeroo Are you using the latest version? We added this feature: https://github.com/woltapp/wolt_modal_sheet/pull/283

sheeroo commented 1 month ago

Hi @sheeroo Are you using the latest version? We added this feature: #283

Hiii, thanks for the quick reply and the amazing work. You are right I was using an older version, sorry for that. I tried this implementation but I was a little disappointed with the result. The drag animation does't follow the finger quite well while dragging down. I think the user experience is quite off. The package smooth_sheets does a really good job with the drag animation if that provides any help. Again thanks for all the work you have put into this :)

ulusoyca commented 1 month ago

@sheeroo Thanks for the input. Would you like to make a contribution? I can support you if you consider.