-
Notifications
You must be signed in to change notification settings - Fork 29.8k
Document ScrollController #9705
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
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.
Scroll controllers
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.
Done
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.
animation => scroll
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.
Done.
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.
widgest => widget
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.
Done.
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.
Maybe "controlled" doesn't bear mentioning here.
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.
Done.
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.
You could try and explain a little more about what would motivate someone to create a custom scroll position. For example it might be worth explaining that PageView uses one to keep the selected page scrolled to the center when the scrollable's viewport size changes.
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.
Done.
|
LGTM |
…9719) * Revert "Document ScrollController (#9705)" This reverts commit ff8609f. * Revert "Fix channels integration test (#9713)" This reverts commit aace622. * Revert "Fix flutter view project's automated tests (#9712)" This reverts commit 1a4bf9b. * Revert "Update hello_world_memory_test package name (#9704)" This reverts commit ffa8483. * Revert "Increment the flutter package version (#9699)" This reverts commit 3b237a8. * Revert "Don’t pod unless there are plugins (#9701)" This reverts commit 732f79e.
No description provided.