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
Reland "Deprecate SingletonFlutterWindow and global window singleton (#39302)" #40511
Conversation
…lutter#39302)" This reverts commit 62f2844.
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.
lib/ui/window.dart
Outdated
/// Deprecated. Will be removed in a future version of Flutter. | ||
/// | ||
/// This global property is deprecated to prepare for Flutter's upcoming support | ||
/// for multiple views and multiple windows. |
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.
/// for multiple views and multiple windows. | |
/// for multiple views and multiple windows. |
/// platform-specific functionality. | ||
/// * [PlatformDispatcher.views], for a list of all available views. | ||
@Deprecated( | ||
'Look up the current FlutterView from the context via View.of(context) or consult the PlatformDispatcher directly instead. ' |
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.
If it's short enough, you could add the method to call on PlatformDispatcher
.
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.
This is already super-long in the analyzer output. I'm gonna skip this :)
Downstream fixes have landed and rolled into the framework in (flutter/flutter#123215). Let's try this again! |
Straight reland without any changes. The fixes were done downstream in flutter/packages#3512 and flutter/flutter#123176.