-
Notifications
You must be signed in to change notification settings - Fork 6k
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
Fix regression in system UI colors #28206
Conversation
if (systemChromeStyle.statusBarIconBrightness != null) { | ||
switch (systemChromeStyle.statusBarIconBrightness) { | ||
case DARK: | ||
// View.SYSTEM_UI_FLAG_LIGHT_STATUS_BAR |
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 value was deprecated in API 30.
Do we know if this still works in Build.VERSION.SDK_INT >= 30 ?
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.
Yeah, the migration should probably be in a separate change though so all of this file can be migrated.
if (systemChromeStyle.systemNavigationBarIconBrightness != null) { | ||
switch (systemChromeStyle.systemNavigationBarIconBrightness) { | ||
case DARK: | ||
// View.SYSTEM_UI_FLAG_LIGHT_NAVIGATION_BAR |
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 one is also deprecated.
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
* 53bc600 Fix regression in system UI colors (flutter/engine#28206) * b423776 Roll Dart SDK from bc7cf49acc5c to 0f5cc29e104b (1 revision) (flutter/engine#28210) * 6057453 Revert "Roll Fuchsia Mac SDK from 9xK8HkMEI... to EZSVwdQoz... (#28200)" (flutter/engine#28214) * 17ddfb3 Revert "Roll Fuchsia Linux SDK from ZSqn1OAt7... to czYvw1Rk3... (#28201)" (flutter/engine#28213)
) * 'Update Dart SDK to aa7d19d' * [web] Don't reset history on hot restart (#27872) * Makes scrollable to use main screen if the flutter view is not attached to a screen (#28110) * Fix regression in system UI colors (#28206) * update licenses golden Co-authored-by: Mouad Debbar <mouad.debbar@gmail.com> Co-authored-by: chunhtai <47866232+chunhtai@users.noreply.github.com> Co-authored-by: Kate Lovett <katelovett@google.com>
…tter#28415) * 'Update Dart SDK to aa7d19d' * [web] Don't reset history on hot restart (flutter#27872) * Makes scrollable to use main screen if the flutter view is not attached to a screen (flutter#28110) * Fix regression in system UI colors (flutter#28206) * update licenses golden Co-authored-by: Mouad Debbar <mouad.debbar@gmail.com> Co-authored-by: chunhtai <47866232+chunhtai@users.noreply.github.com> Co-authored-by: Kate Lovett <katelovett@google.com>
Fixes flutter/flutter#88431
This fixes a regression introduced in #27018. The refactored style portion made color changes dependent on brightness not being null.
Pre-launch Checklist
writing and running engine tests.
///
).If you need help, consider asking for advice on the #hackers-new channel on Discord.