Skip to content

Improve splash - #17688

Merged
alperozturk96 merged 4 commits into
masterfrom
improve-splash
Sep 25, 2026
Merged

alperozturk96 merged 4 commits into
masterfrom
improve-splash

Conversation

@alperozturk96

@alperozturk96 alperozturk96 commented Sep 22, 2026 •

Copy link
Copy Markdown
Collaborator

Changes

  • Uses app logo for splash
  • Removes any hardcoded delay for non-branding flow
  • No need extend from BaseActivity

Current behavior (master)

When the app starts, the splash screen has two phases:

  1. The system shows the splash screen with a blue background.
  2. The Activity shows its layout. An ImageView in 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 from ic_foreground and ic_background. You can set a different drawable in windowSplashScreenAnimatedIcon. This does not change the launcher icon but this means brander must override values for this as well.


🏚️ Master (1) 🏡 After
1.mp4
2.mp4

@github-actions

github-actions Bot commented Sep 22, 2026 •

Copy link
Copy Markdown

📱 QA build

Download app-qa-debug.apk
QR code Open the QR code for this download
Commit 381e432
Version 17688
Available until 7 days after this build

The QA build installs alongside a released Nextcloud app, so you can keep
using your existing install while testing.

Downloading the file requires a GitHub account, so open this link on the
device you want to test on, or transfer the APK to it.

@github-actions

Copy link
Copy Markdown

stable22-IT test failed, but no output was generated. Maybe a preliminary stage failed.

@github-actions

Copy link
Copy Markdown

master-IT test failed, but no output was generated. Maybe a preliminary stage failed.

@codecov

codecov Bot commented Sep 22, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 66.66667% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...c/main/java/com/nmc/android/ui/LauncherActivity.kt 66.66% 1 Missing and 2 partials ⚠️

📢 Thoughts on this report? Let us know!

@AndyScherzinger

Copy link
Copy Markdown
Member

LGTM, one questions though is it still possible to set the headline and subline texts in general, not regarding this PR?

@alperozturk96

Copy link
Copy Markdown
Collaborator Author

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?

@alperozturk96 alperozturk96 added this to the Nextcloud App 35.0.1 milestone Sep 25, 2026
@alperozturk96

Copy link
Copy Markdown
Collaborator Author

/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>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
@github-actions

Copy link
Copy Markdown

Codacy

SpotBugs

CategoryBaseNew
Bad practice3232
Correctness5858
Dodgy code185185
Experimental11
Malicious code vulnerability22
Multithreaded correctness2626
Performance3737
Security1717
Total358358

@github-actions

Copy link
Copy Markdown

@alperozturk96
alperozturk96 merged commit 32b474f into master Sep 25, 2026
22 of 24 checks passed
@alperozturk96
alperozturk96 deleted the improve-splash branch September 25, 2026 08:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants