Skip to content
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

Improve splash display on android #3105

Merged

Conversation

edo-bari-ikutsu
Copy link
Contributor

The current Android app displays a white screen before the splash screen. A white status bar is also displayed. I think this white display degrades the user experience of the app (especially on dark mode).

current_light current_dark

By applying this fix, white display is gone and startup and splash become more continuous and natural.

pr_light pr_dark

Note: this fix is only for Android apps, as I don't have iOS devices. To avoid side effects to the iOS app, I have applied some conditional branches and code separation between Android and iOS.

  • Example 1: Apart from the existing SPLASH_CONFIG and DARK_SPLASH_CONFIG in app.config.js,
    SPLASH_CONFIG_ANDROID and DARK_SPLASH_CONFIG_ANDROID added.
  • Example 2: <StatusBar /> in App.native.tsx is only added on Android.

@surfdude29
Copy link
Contributor

surfdude29 commented Mar 5, 2024

Fixes #2292.

Also fwiw I think the white screen on startup is only an issue on Android. In my experience, iOS displays the splash image (starry sky or blue sky) immediately after opening the app. I also couldn't find any issues reporting the white screen pre-splash on iOS.

@haileyok haileyok self-assigned this Mar 6, 2024
@haileyok
Copy link
Contributor

haileyok commented Mar 6, 2024

Another awesome PR!

Yup, this isn't an issue on iOS. Wasn't aware of this expo config plugin at all, thank you so much for sharing!

Copy link
Contributor

@haileyok haileyok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@haileyok haileyok merged commit 26fc0cf into bluesky-social:main Mar 6, 2024
4 checks passed
@haileyok
Copy link
Contributor

haileyok commented Mar 6, 2024

@edo-bari-ikutsu By the way, this is definitely something that might be worth bringing up in the @expo repo if you feel like it. Seems to be something that a ton of people run into!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants