Skip to content

[PageTransitionsBuilder] Reland Fix 'ZoomPageTransition' built more than once - #60245

Merged
fluttergithubbot merged 1 commit into
masterfrom
revert-59992-revert-58686-fix-zoom-page-transition-build
Jun 25, 2020
Merged

fluttergithubbot merged 1 commit into
masterfrom
revert-59992-revert-58686-fix-zoom-page-transition-build

Conversation

@shihaohong

@shihaohong shihaohong commented Jun 24, 2020 •

Copy link
Copy Markdown
Contributor

Description

This has been unblocked internally via cl/318079801. b/159779895 was added to track fixing the tests that were originally preventing #58686 from landing.

cc/ @AlexVincent525

Issues

Fixes #58686
Reverts #59992

@shihaohong
shihaohong requested review from goderbauer and renyou June 24, 2020 23:03
@fluttergithubbot fluttergithubbot added p: material_ui material_ui package in flutter/packages framework flutter/packages/flutter repository. See also f: labels. labels Jun 24, 2020

@renyou renyou left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@flutter-github-sync

Copy link
Copy Markdown

Started Google testing for this PR

@shihaohong

Copy link
Copy Markdown
Contributor Author

@renyou Unrelated to the PR itself, but does/should the waiting for tree to go green label account for Google testing for the PR? I was thinking of the case where this PR merges before we get a result from Google testing.

@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

@renyou

renyou commented Jun 24, 2020 via email

Copy link
Copy Markdown
Contributor

@fluttergithubbot
fluttergithubbot merged commit 17cb865 into master Jun 25, 2020
@fluttergithubbot
fluttergithubbot deleted the revert-59992-revert-58686-fix-zoom-page-transition-build branch June 25, 2020 00:03
@flutter-github-sync

Copy link
Copy Markdown

Google testing passed!

@shihaohong shihaohong changed the title Reland [PageTransitionsBuilder] Fix 'ZoomPageTransition' built more than once [PageTransitionsBuilder] Reland Fix 'ZoomPageTransition' built more than once Jun 25, 2020
mingwandroid pushed a commit to mingwandroid/flutter that referenced this pull request Sep 6, 2020
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jul 30, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

framework flutter/packages/flutter repository. See also f: labels. p: material_ui material_ui package in flutter/packages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants