Skip to content

Limit overscroll stretching - #99364

Merged
fluttergithubbot merged 1 commit into
flutter:masterfrom
Piinks:stretchEdge
Mar 2, 2022
Merged

Limit overscroll stretching#99364
fluttergithubbot merged 1 commit into
flutter:masterfrom
Piinks:stretchEdge

Conversation

@Piinks

@Piinks Piinks commented Mar 2, 2022

Copy link
Copy Markdown
Contributor

Fixes #99264

This limits the amount of stretch that the indicator can reach, regardless of how many pointers you apply.
(I see you #11884 😝 )

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
  • I signed the CLA.
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or this PR is test-exempt.
  • All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@Piinks Piinks added platform-android Android applications specifically framework flutter/packages/flutter repository. See also f: labels. p: material_ui material_ui package in flutter/packages a: fidelity Matching the OEM platforms better f: scrolling Viewports, list views, slivers, etc. labels Mar 2, 2022
@Piinks
Piinks requested a review from goderbauer March 2, 2022 00:25
goderbauer
goderbauer approved these changes Mar 2, 2022
View reviewed changes

@goderbauer goderbauer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@fluttergithubbot

fluttergithubbot commented Mar 2, 2022

Copy link
Copy Markdown
Contributor

This pull request is not suitable for automatic merging in its current state.

  • The status or check suite Google testing has failed. Please fix the issues identified (or deflake) before re-applying this label.

@Piinks

Piinks commented Mar 2, 2022

Copy link
Copy Markdown
Contributor Author

There are some approved customer screenshot tests that capture the stretch changing here. Updating these is expected. Setting the google testing shard to green manually to land.

@fluttergithubbot
fluttergithubbot merged commit d211f3f into flutter:master Mar 2, 2022
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Mar 2, 2022
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 2, 2022
@engine-flutter-autoroll
d211f3f Limit overscroll stretching (flutter/flutter#99364)
b185240
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Mar 2, 2022
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 2, 2022
@engine-flutter-autoroll
d211f3f Limit overscroll stretching (flutter/flutter#99364)
1447e53
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Mar 2, 2022
Merged
engine-flutter-autoroll added a commit to engine-flutter-autoroll/plugins that referenced this pull request Mar 2, 2022
@engine-flutter-autoroll
d211f3f Limit overscroll stretching (flutter/flutter#99364)
2a35f68
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Mar 2, 2022
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 2, 2022
@engine-flutter-autoroll
d211f3f Limit overscroll stretching (flutter/flutter#99364)
0ed3cb4
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Mar 3, 2022
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 3, 2022
@engine-flutter-autoroll
d211f3f Limit overscroll stretching (flutter/flutter#99364)
5318b73
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Mar 3, 2022
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 3, 2022
@engine-flutter-autoroll
d211f3f Limit overscroll stretching (flutter/flutter#99364)
865076b
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Mar 3, 2022
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 3, 2022
@engine-flutter-autoroll
d211f3f Limit overscroll stretching (flutter/flutter#99364)
d40d495
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Mar 3, 2022
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 3, 2022
@engine-flutter-autoroll
d211f3f Limit overscroll stretching (flutter/flutter#99364)
1149f89
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 4, 2022
@engine-flutter-autoroll
d211f3f Limit overscroll stretching (flutter/flutter#99364)
a04c539
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Mar 4, 2022
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 4, 2022
@engine-flutter-autoroll
d211f3f Limit overscroll stretching (flutter/flutter#99364)
8de5b54
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Mar 4, 2022
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 4, 2022
@engine-flutter-autoroll
d211f3f Limit overscroll stretching (flutter/flutter#99364)
ff67016
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Mar 5, 2022
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 5, 2022
@engine-flutter-autoroll
d211f3f Limit overscroll stretching (flutter/flutter#99364)
054995a
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Mar 5, 2022
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 5, 2022
@engine-flutter-autoroll
d211f3f Limit overscroll stretching (flutter/flutter#99364)
a4c846e
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Mar 5, 2022
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 5, 2022
@engine-flutter-autoroll
d211f3f Limit overscroll stretching (flutter/flutter#99364)
a51c184
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Mar 5, 2022
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 5, 2022
@engine-flutter-autoroll
d211f3f Limit overscroll stretching (flutter/flutter#99364)
e68c214
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Mar 5, 2022
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 5, 2022
@engine-flutter-autoroll
d211f3f Limit overscroll stretching (flutter/flutter#99364)
07ab4bf
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Mar 5, 2022
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 5, 2022
@engine-flutter-autoroll
d211f3f Limit overscroll stretching (flutter/flutter#99364)
d44ad6f
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Mar 5, 2022
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 5, 2022
@engine-flutter-autoroll
d211f3f Limit overscroll stretching (flutter/flutter#99364)
ed81733
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Mar 5, 2022
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 5, 2022
@engine-flutter-autoroll
d211f3f Limit overscroll stretching (flutter/flutter#99364)
902bac0
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Mar 5, 2022
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 5, 2022
@engine-flutter-autoroll
d211f3f Limit overscroll stretching (flutter/flutter#99364)
7c6e226
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Mar 5, 2022
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 5, 2022
@engine-flutter-autoroll
d211f3f Limit overscroll stretching (flutter/flutter#99364)
9d9687d
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Mar 5, 2022
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 5, 2022
@engine-flutter-autoroll
d211f3f Limit overscroll stretching (flutter/flutter#99364)
caaf294
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Mar 5, 2022
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 5, 2022
@engine-flutter-autoroll
d211f3f Limit overscroll stretching (flutter/flutter#99364)
7e691e5
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 5, 2022
@engine-flutter-autoroll
d211f3f Limit overscroll stretching (flutter/flutter#99364)
6fa893f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@goderbauer goderbauer goderbauer approved these changes

Assignees

No one assigned

Labels

a: fidelity Matching the OEM platforms better f: scrolling Viewports, list views, slivers, etc. framework flutter/packages/flutter repository. See also f: labels. p: material_ui material_ui package in flutter/packages platform-android Android applications specifically

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Android] Lists with AndroidOverscrollIndicator.stretch can be over scrolled infinitely with multi-touch

3 participants

@Piinks @fluttergithubbot @goderbauer