Skip to content

Fix scrollable touch scrolling when out of bounds#2906

Merged
hecrj merged 1 commit intoiced-rs:masterfrom
lmaxyz:master
Nov 25, 2025
Merged

Fix scrollable touch scrolling when out of bounds#2906
hecrj merged 1 commit intoiced-rs:masterfrom
lmaxyz:master

Conversation

@lmaxyz
Copy link
Contributor

@lmaxyz lmaxyz commented Apr 27, 2025

I ran into problem with scrollable by touch. The scrollable widget scrolled even when touch wasn't under the scroll area. This patch fixes the problem. And as I see it doesn't have side effects.

@lmaxyz
Copy link
Contributor Author

lmaxyz commented May 7, 2025

Here is a demo, how fix is looking on device.
Left device has fix, and right one hasn't and scrollable widget works incorrectly. It scrolls content even if touch starts not on the scrollable area.

iced_fix_demo.mp4

@hecrj hecrj added this to the 0.14 milestone Nov 25, 2025
@hecrj hecrj added bug Something isn't working widget fix labels Nov 25, 2025
Copy link
Member

@hecrj hecrj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@hecrj hecrj enabled auto-merge November 25, 2025 20:42
@hecrj hecrj changed the title Fix scrollable scrolling by touch Fix scrollable touch scrolling when out of bounds Nov 25, 2025
@hecrj hecrj merged commit 45aeccc into iced-rs:master Nov 25, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working fix widget

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants