-
Notifications
You must be signed in to change notification settings - Fork 29.7k
[flutter roll] Revert "Add ability for ModalRoutes to ignore pointers during trans…
#104520
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
…itions and do so on `Cupertino` routes (flutter#95757)" This reverts commit 4c0b0be.
chunhtai
left a comment
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.
LGTM
|
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. |
goderbauer
left a comment
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.
LGTM
ModalRoutes to ignore pointers during trans…ModalRoutes to ignore pointers during trans…
…ng transitions and do so on `Cupertino` routes (#95757)" (flutter/flutter#104520)
…itions and do so on `Cupertino` routes (flutter#95757)" (flutter#104520) This reverts commit 4c0b0be.
…ng transitions and do so on `Cupertino` routes (#95757)" (flutter/flutter#104520)
|
@XilaiZhang what was the issue here? Anything I can help with? |
|
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 |
…itions and do so on `Cupertino` routes (flutter#95757)" (flutter#104520) This reverts commit 4c0b0be.
…ng transitions and do so on `Cupertino` routes (#95757)" (flutter/flutter#104520)
…ng transitions and do so on `Cupertino` routes (#95757)" (flutter/flutter#104520)
…itions and do so on
Cupertinoroutes (#95757)"This reverts commit 4c0b0be.