Skip to content

Conversation

@abarth
Copy link
Contributor

@abarth abarth commented May 1, 2017

No description provided.

@abarth abarth requested review from HansMuller and Hixie May 1, 2017 21:18
Copy link
Contributor

Choose a reason for hiding this comment

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

Scroll controllers

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

Copy link
Contributor

Choose a reason for hiding this comment

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

animation => scroll

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

Copy link
Contributor

Choose a reason for hiding this comment

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

widgest => widget

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

Copy link
Contributor

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

Copy link
Contributor

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

@HansMuller
Copy link
Contributor

LGTM

@abarth abarth force-pushed the scroll_controller branch from 62a7c76 to b7e6a76 Compare May 1, 2017 23:19
@abarth abarth force-pushed the scroll_controller branch from b7e6a76 to ecd47c3 Compare May 2, 2017 00:16
@abarth abarth merged commit ff8609f into flutter:master May 2, 2017
@abarth abarth deleted the scroll_controller branch May 2, 2017 02:18
Hixie added a commit that referenced this pull request May 2, 2017
Hixie added a commit that referenced this pull request May 2, 2017
…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.
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 13, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants