Skip to content

Conversation

@XilaiZhang
Copy link
Contributor

…itions and do so on Cupertino routes (#95757)"

This reverts commit 4c0b0be.

…itions and do so on `Cupertino` routes (flutter#95757)"

This reverts commit 4c0b0be.
@flutter-dashboard flutter-dashboard bot added a: tests "flutter test", flutter_test, or one of our tests f: cupertino flutter/packages/flutter/cupertino repository f: material design flutter/packages/flutter/material repository. f: routes Navigator, Router, and related APIs. framework flutter/packages/flutter repository. See also f: labels. labels May 24, 2022
Copy link
Contributor

@chunhtai chunhtai left a comment

Choose a reason for hiding this comment

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

LGTM

@goderbauer
Copy link
Member

Please briefly add some context why this was reverted for people who want to look at this again in the future and cc the original author for their information that it got reverted.

Copy link
Member

@goderbauer goderbauer left a comment

Choose a reason for hiding this comment

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

LGTM

@chunhtai chunhtai changed the title [flutter g3 roll] Revert "Add ability for ModalRoutes to ignore pointers during trans… [flutter roll] Revert "Add ability for ModalRoutes to ignore pointers during trans… May 24, 2022
@XilaiZhang XilaiZhang merged commit de230d3 into flutter:master May 24, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 24, 2022
…ng transitions and do so on `Cupertino` routes (#95757)" (flutter/flutter#104520)
XilaiZhang added a commit to XilaiZhang/flutter that referenced this pull request May 24, 2022
…itions and do so on `Cupertino` routes (flutter#95757)" (flutter#104520)

This reverts commit 4c0b0be.
engine-flutter-autoroll added a commit to engine-flutter-autoroll/plugins that referenced this pull request May 24, 2022
…ng transitions and do so on `Cupertino` routes (#95757)" (flutter/flutter#104520)
CaseyHillers pushed a commit that referenced this pull request May 24, 2022
…itions and do so on `Cupertino` routes (#95757)" (#104520) (#104532)

This reverts commit 4c0b0be.
@willlockwood
Copy link
Contributor

willlockwood commented May 25, 2022

@XilaiZhang what was the issue here? Anything I can help with?

@XilaiZhang
Copy link
Contributor Author

yes sir i copied the error msg to #95757 (comment). I am not sure if there is an open source version of the test you can reference

@XilaiZhang XilaiZhang deleted the xilaizhang-revert-1 branch July 6, 2022 16:47
camsim99 pushed a commit to camsim99/flutter that referenced this pull request Aug 10, 2022
…itions and do so on `Cupertino` routes (flutter#95757)" (flutter#104520)

This reverts commit 4c0b0be.
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Aug 30, 2022
…ng transitions and do so on `Cupertino` routes (#95757)" (flutter/flutter#104520)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/plugins that referenced this pull request Aug 30, 2022
…ng transitions and do so on `Cupertino` routes (#95757)" (flutter/flutter#104520)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: tests "flutter test", flutter_test, or one of our tests f: cupertino flutter/packages/flutter/cupertino repository f: material design flutter/packages/flutter/material repository. f: routes Navigator, Router, and related APIs. framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants