Skip to content
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

Moved two animation vignettes into misc. #425

Merged
merged 2 commits into from May 4, 2020

Conversation

@RedBrogdon
Copy link
Contributor

RedBrogdon commented Apr 30, 2020

Moves the vignettes featuring AnimatedPositioned and AnimatedSwitcher from basics to misc.

The basics section is designed to follow a particular progression of topics, easing the reader into more and more advanced bits of the animations API. For that reason, these two examples are better suited to the miscellaneous section.

@RedBrogdon RedBrogdon requested a review from johnpryan Apr 30, 2020
@AyushBherwani1998
Copy link
Contributor

AyushBherwani1998 commented May 1, 2020

@RedBrogdon we can also update the README.md and move the examples from Basics to Misc section.

@johnpryan
Copy link
Contributor

johnpryan commented May 1, 2020

That's a good point, we need to update the README too.

@jddeep
Copy link
Contributor

jddeep commented May 2, 2020

@RedBrogdon we can also update the README.md and move the examples from Basics to Misc section.

👍

@RedBrogdon
Copy link
Contributor Author

RedBrogdon commented May 4, 2020

Yep, should have updated the README. I've pushed another commit.

@johnpryan, I saw you approved, but do you mind taking another quick look? I reformatted a couple things and added a codelab link. Just want to make sure it's all good by you.

Copy link
Contributor

johnpryan left a comment

LGTM

@RedBrogdon RedBrogdon merged commit c7f5030 into flutter:master May 4, 2020
2 of 3 checks passed
2 of 3 checks passed
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
WIP Ready for review
Details
cla/google All necessary CLAs are signed
@RedBrogdon RedBrogdon deleted the RedBrogdon:animations_moves branch May 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.