-
-
Notifications
You must be signed in to change notification settings - Fork 51
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
Remove deprecated Android SplashScreenDrawable
.
#539
Conversation
We still used an deprecated way to declare a splash screen, which caused the following warning to be printed on Android start-up: `W/FlutterActivityAndFragmentDelegate( 6179): A splash screen was provided to Flutter, but this is deprecated. See flutter.dev/go/android-splash-migration for migration steps.` Since we also already use the recommand way to declare a splash screen additionally, we just had to remove the old way. There are no behavior / splash screen changes, everything works as it used to.
To view this pull requests documentation preview, visit the following URL: docs.page/sharezoneapp/sharezone-app~539 Documentation is deployed and generated using docs.page. |
Visit the preview URL for this PR (updated for commit d347bac): https://sharezone-test--pr539-remove-deprecated-an-3z8847fl.web.app (expires Thu, 17 Aug 2023 00:26:04 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 4cb3ae61e1e018abfd9841fd3239f5b49ccc034b |
⬇️ Generated builds by Codemagic for commit d347bac ⬇️ Note: Only Sharezone team members are able to install the iOS app.
|
…roid-splash-screen
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
We still used an deprecated way to declare a splash screen, which caused the following warning to be printed on Android start-up:
W/FlutterActivityAndFragmentDelegate( 6179): A splash screen was provided to Flutter, but this is deprecated. See flutter.dev/go/android-splash-migration for migration steps.
Since we also already use the recommended way to declare a splash screen additionally, we just had to remove the old way. There are no behavior / splash screen changes, everything works as it used to.