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

Splashscreen implemented #189

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Splashscreen implemented #189

wants to merge 5 commits into from

Conversation

stefan-niedermann
Copy link
Contributor

@stefan-niedermann stefan-niedermann commented Apr 11, 2018

I implemented a splashscreen like in Notes, News and (soon) Files app.

Attention: This will remove the launcher icon from home screens when the user updates the app (basically the activity changes). With the last commit i managed to keep the old launcher - for the user nothing will change :)

I think it might be possible to keep the launchers on the homescreen, if we keep the names, but it will be weird in your code when your MainActivity launches the SplashscreenActivity and the RealMainActivity launches the MainActivity - that's why i implemented it clean ;-) Got some better inspiration by https://github.com/nextcloud/android/pull/2448/files

screenshot_1523429831

@stefan-niedermann
Copy link
Contributor Author

Good news @nerzhul,
i got some inspiration by https://github.com/nextcloud/android/pull/2448/files to keep the former launcher on the homescreen.

So i think this PR is safe to merge without breaking changes for the users :)

@stefan-niedermann
Copy link
Contributor Author

@nerzhul any concerns about this PR? Can i provide more information or help you?

@AndyScherzinger
Copy link
Contributor

pinging @nerzhul about this open PR ❤️

@nerzhul
Copy link
Owner

nerzhul commented Nov 18, 2018

i'm not sure it's useful, app load very fast

@stefan-niedermann
Copy link
Contributor Author

Do you see disadvantages?

I think there are three reasons why this should be merged:

  1. There is no artificial timeout or something like this - the splash screen does not slow down the app start but shows while the natural start process
  2. Nearly all nextcloud ecosystem apps are doing this the same way - user will feel more comfortable, more like being "at home" when all the Nextcloud apps will have a similiar look and feel (just like the story about the adaptive icons, you remember?)
  3. Think about older or cheaper devices. Sure, on your device it might work fast, but there are a plenty of users with old weak or cheap phones, where many apps are running in the background

In short: you don't know in which environement your app will run. So if there are no disadvantages for you, but there mght be advantages fore others, why not?

@e-alfred
Copy link
Contributor

e-alfred commented Jun 3, 2019

@nerzhul currently has not much time for the Nextcloud apps. Maybe @nerzhul you can add @stefan-niedermann as a maintainer to this repository/app?

@stefan-niedermann It would be awesome if we could get the Nextcloud SMS app back on track again. Would you be interested in maintaining the client?

@stefan-niedermann
Copy link
Contributor Author

I am currently very busy with the Notes and Deck android app, therefore i don't have much time unfortunately.

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.

4 participants