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
Add a warning to stop launching on pub for DevTools version 2.8.0 #3464
Conversation
return Material( | ||
color: colorScheme.overlayShadowColor, | ||
child: Center( | ||
child: Container( |
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.
nit: can center be moved into container? Container can align to center.
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 some reason, this makes the Container full screen instead of the size of its content.
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.
The
showPubWarning
flag will only be turned on a for a special release of DevTools 2.8.0 to pub. This ensures we will not mistakenly show this warning for DevTools 2.8.0 shipped in the SDK. When DevTools is launched from pub for a user that is on Flutter >= 2.8.0 or Dart >= 2.15.0, this overlay warning will show when the user opens DevTools and connects to their app.This warning will also show in the About DevTools menu.
Fixes #3433