Skip to content

[stable-3.24] Use adaptive colors for pre-login screens (#11362)#11376

Merged
AlvaroBrey merged 1 commit intostable-3.24from
backport/11362/stable-3.24-manual
Feb 13, 2023
Merged

[stable-3.24] Use adaptive colors for pre-login screens (#11362)#11376
AlvaroBrey merged 1 commit intostable-3.24from
backport/11362/stable-3.24-manual

Conversation

@AlvaroBrey
Copy link
Member

Manual backport of #11362

* Use unmodified primary color as background through M3 utils for login screen

Signed-off-by: Álvaro Brey <alvaro.brey@nextcloud.com>

* Use color utils to fix colors in AuthenticatorActivity for light primaries

Signed-off-by: Álvaro Brey <alvaro.brey@nextcloud.com>

* WhatsNewActivity: fix for light primary color, port to viewbinding

Signed-off-by: Álvaro Brey <alvaro.brey@nextcloud.com>

* Fix status bar colors in pre-login screens

Signed-off-by: Álvaro Brey <alvaro.brey@nextcloud.com>

* FeatureFragment: fix spotbugs

Signed-off-by: Álvaro Brey <alvaro.brey@nextcloud.com>

* Update login screenshot

Signed-off-by: Álvaro Brey <alvaro.brey@nextcloud.com>

* build: Set ui dep to 0.5.0 tag

Signed-off-by: Álvaro Brey <alvaro.brey@nextcloud.com>

---------

Signed-off-by: Álvaro Brey <alvaro.brey@nextcloud.com>
@github-actions
Copy link

Lint

Typestable-3.24PR
Warnings7676
Errors00

SpotBugs

CategoryBaseNew
Bad practice2727
Correctness4444
Dodgy code334331
Internationalization99
Multithreaded correctness99
Performance5858
Security1818
Total499496

@github-actions
Copy link

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/11376.apk

qrcode

To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.

@codecov
Copy link

codecov bot commented Feb 13, 2023

Codecov Report

Merging #11376 (cebf953) into stable-3.24 (5828227) will increase coverage by 0.11%.
The diff coverage is 42.62%.

Additional details and impacted files
@@                Coverage Diff                @@
##             stable-3.24   #11376      +/-   ##
=================================================
+ Coverage          31.41%   31.53%   +0.11%     
- Complexity          3404     3413       +9     
=================================================
  Files                575      575              
  Lines              42018    42028      +10     
  Branches            5660     5661       +1     
=================================================
+ Hits               13202    13252      +50     
+ Misses             26871    26830      -41     
- Partials            1945     1946       +1     
Impacted Files Coverage Δ
.../nextcloud/client/onboarding/WhatsNewActivity.java 0.00% <0.00%> (ø)
.../owncloud/android/ui/fragment/FeatureFragment.java 34.48% <30.00%> (-0.61%) ⬇️
...wncloud/android/ui/whatsnew/ProgressIndicator.java 69.04% <75.00%> (+0.75%) ⬆️
.../nextcloud/client/onboarding/FirstRunActivity.java 36.78% <100.00%> (ø)
.../android/authentication/AuthenticatorActivity.java 40.24% <100.00%> (+0.63%) ⬆️
...android/utils/theme/MaterialSchemesProviderImpl.kt 95.65% <100.00%> (+0.41%) ⬆️
...com/owncloud/android/utils/theme/ViewThemeUtils.kt 92.30% <100.00%> (+33.97%) ⬆️
.../java/com/nextcloud/client/jobs/OfflineSyncWork.kt 20.31% <0.00%> (-40.63%) ⬇️
...d/android/operations/SynchronizeFileOperation.java 24.46% <0.00%> (-12.77%) ⬇️
.../third_parties/daveKoeller/AlphanumComparator.java 86.90% <0.00%> (-1.20%) ⬇️
... and 13 more

@AlvaroBrey AlvaroBrey merged commit 7fdebd8 into stable-3.24 Feb 13, 2023
@delete-merged-branch delete-merged-branch bot deleted the backport/11362/stable-3.24-manual branch February 13, 2023 13:14
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.

1 participant

Comments