New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update Scrollbar behavior for mobile devices #72531
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
// Hover events not supported on mobile. | ||
child = MouseRegion( | ||
onExit: handleHoverExit, | ||
onHover: handleHover, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can use PointerEvent.kind
to distinguish the type of input device to deal with this issue. Moreover, the mobile phone can also be used with an external mouse. At this time, we can better respond to the hover event.
Description
Follow-up change to #71664
The new material scrollbar needs a few mobile-specific updates.
First, the scrollbar should be thinner on mobile because of the relative size of the screen.
On Android specifically, we should be matching native instead of Material Design.
Finally, on mobile, we should be ignoring hover events. You can trigger a hover enter event with taps on mobile, but you cannot trigger the hover exit event, which the behaviors of the scrollbar relies on.
Related Issues
Fixes #72202
#70866
Tests
Updated affected tests, added tests for platform specific behaviors.
Checklist
Before you create this PR, confirm that it meets all requirements listed below by checking the relevant checkboxes (
[x]
). This will ensure a smooth and quick review process.///
).flutter analyze --flutter-repo
) does not report any problems on my PR.Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.