Improve splash - #17688
Improve splash#17688
Conversation
📱 QA build
The QA build installs alongside a released Nextcloud app, so you can keep Downloading the file requires a GitHub account, so open this link on the |
|
stable22-IT test failed, but no output was generated. Maybe a preliminary stage failed. |
|
master-IT test failed, but no output was generated. Maybe a preliminary stage failed. |
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
25ff38a to
bead27c
Compare
|
LGTM, one questions though is it still possible to set the headline and subline texts in general, not regarding this PR? |
Yes those are didn't deleted https://github.com/nextcloud/android/pull/17688/changes#diff-49701c3de3f691e664e4da47ddfc141e08cf057439becc327dd4a6b00f696d57L33 @tobiasKaminsky Can I test that flow as well from brander for this PR? Or manually configuring setup.xml is enough? |
|
/backport to stable-35.0.x |
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
efefea4 to
381e432
Compare
|
stable35-IT test failed: https://www.kaminsky.me/nc-dev/android-integrationTests/17688-IT-stable35-08-31 |
Changes
BaseActivityCurrent behavior (master)
When the app starts, the splash screen has two phases:
ImageViewin this layout shows the logo with the text "Nextcloud".The splash icon and the launcher icon are different items. If you do not set a splash icon, the system uses
@mipmap/ic_launcher. This icon is made fromic_foregroundandic_background. You can set a different drawable inwindowSplashScreenAnimatedIcon. This does not change the launcher icon but this means brander must override values for this as well.1.mp4
2.mp4