Skip to content

Reapply new PopScope API - #147607

Merged
chunhtai merged 5 commits into
flutter:masterfrom
chunhtai:reland-popscope
May 7, 2024
Merged

chunhtai merged 5 commits into
flutter:masterfrom
chunhtai:reland-popscope

Conversation

@chunhtai

Copy link
Copy Markdown
Contributor

same as previous but with soft transition on modalroute

Pre-launch Checklist

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@chunhtai
chunhtai requested a review from justinmc April 30, 2024 22:19
@github-actions github-actions Bot added a: text input Entering text in a text field or keyboard related problems framework flutter/packages/flutter repository. See also f: labels. p: material_ui material_ui package in flutter/packages p: cupertino_ui cupertino_ui package in flutter/packages d: api docs Issues with https://api.flutter.dev/ d: examples Sample code and demos f: routes Navigator, Router, and related APIs. labels Apr 30, 2024

@justinmc justinmc 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 👍 Thanks for making this easy to review with your 2 separate commits.

What was the problem exactly? So #147016 was reverted in #147597 due to flutter_eval, but why did Route have to be migrated as well?

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.

The analyzer says you're missing a ; here.

@chunhtai

chunhtai commented May 1, 2024

Copy link
Copy Markdown
Contributor Author

The flutter_eval overrides PageRoute.onpopInvoke https://github.com/ethanblake4/flutter_eval/blob/a4c354b134bb26b7a9594bcb5866923a1b707917/lib/src/widgets/pages.dart#L224

@chunhtai
chunhtai force-pushed the reland-popscope branch from 1872e69 to f7dc64a Compare May 1, 2024 18:41
@chunhtai
chunhtai force-pushed the reland-popscope branch from 15d4e76 to ec26457 Compare May 6, 2024 19:05
@chunhtai
chunhtai merged commit 007faa9 into flutter:master May 7, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 8, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 8, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 8, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 8, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Aug 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: text input Entering text in a text field or keyboard related problems d: api docs Issues with https://api.flutter.dev/ d: examples Sample code and demos f: routes Navigator, Router, and related APIs. framework flutter/packages/flutter repository. See also f: labels. p: cupertino_ui cupertino_ui package in flutter/packages p: material_ui material_ui package in flutter/packages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants