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

Prevent white splash on cold start #868

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tophf
Copy link

@tophf tophf commented Dec 20, 2018

Currently gh4a sets the user theme in BaseActivity::onCreate()

protected void onCreate(Bundle savedInstanceState) {
setTheme(Gh4Application.THEME);
but still the white splash is frequently shown on budget devices during a cold start of the app, which is extremely annoying in case the dark theme is used in the app and in the entire UI of the device (lots of apps have this problem, BTW).

This patch makes it so that a transparent default theme is shown before the actual theme is applied. On a slower device nothing will be shown after the user launched the app until the main screen is presented so one might think that could perplex some users, but I think they are used to their device being slow and encounter such pauses regularly. On a fast device there'll be no perceivable difference to the old behavior.

Currently gh4a sets the user theme in BaseActivity::onCreate() but still the white splash is frequently shown on budget devices during a cold start of the app, which is extremely annoying in case the dark theme is used in the app and in the entire UI of the device.

This patch makes it so that a transparent default theme is shown before the actual theme is applied. On a slower device nothing will be shown after the user launched the app until the main screen is presented so one might think that could perplex some users, but I think they are used to their device being slow and encounter such pauses regularly. On a fast device there'll be no perceivable difference to the old behavior.
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.

None yet

1 participant