Skip to content

Fix white header with dark mode#5107

Merged
tobiasKaminsky merged 1 commit into
masterfrom
whiteHeader
Jan 8, 2020
Merged

Fix white header with dark mode#5107
tobiasKaminsky merged 1 commit into
masterfrom
whiteHeader

Conversation

@tobiasKaminsky

@tobiasKaminsky tobiasKaminsky commented Jan 7, 2020

Copy link
Copy Markdown
Member

Fix #4754

2020-01-07-092854 2020-01-07-092844

White background of header will not be changed, as this is desired via theming api.
But at least we can now see the text again ;-)

(also applies for all other activities)

Signed-off-by: tobiasKaminsky tobias@kaminsky.me

Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
@nextcloud-android-bot

Copy link
Copy Markdown
Collaborator

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/12146.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.

@nextcloud-android-bot

Copy link
Copy Markdown
Collaborator

Codacy

329

Lint

TypemasterPR
Warnings7474
Errors00

SpotBugs (new)

Warning TypeNumber
Bad practice Warnings26
Correctness Warnings71
Internationalization Warnings13
Malicious code vulnerability Warnings5
Multithreaded correctness Warnings9
Performance Warnings106
Security Warnings44
Dodgy code Warnings138
Total412

SpotBugs (master)

Warning TypeNumber
Bad practice Warnings26
Correctness Warnings71
Internationalization Warnings13
Malicious code vulnerability Warnings5
Multithreaded correctness Warnings9
Performance Warnings106
Security Warnings44
Dodgy code Warnings138
Total412

@codecov

codecov Bot commented Jan 7, 2020

Copy link
Copy Markdown

Codecov Report

Merging #5107 into master will decrease coverage by <.01%.
The diff coverage is 58.33%.

@@             Coverage Diff              @@
##             master    #5107      +/-   ##
============================================
- Coverage     17.79%   17.78%   -0.01%     
  Complexity        3        3              
============================================
  Files           389      389              
  Lines         32890    32893       +3     
  Branches       4626     4627       +1     
============================================
- Hits           5852     5850       -2     
- Misses        26082    26087       +5     
  Partials        956      956
Impacted Files Coverage Δ Complexity Δ
...owncloud/android/ui/activity/SettingsActivity.java 39.57% <100%> (ø) 0 <0> (ø) ⬇️
.../owncloud/android/ui/activity/ToolbarActivity.java 63.76% <100%> (ø) 0 <0> (ø) ⬇️
...in/java/com/owncloud/android/utils/ThemeUtils.java 49.23% <42.85%> (-1.75%) 0 <0> (ø)
...m/owncloud/android/ui/activity/DrawerActivity.java 45.17% <66.66%> (ø) 0 <0> (ø) ⬇️
.../third_parties/daveKoeller/AlphanumComparator.java 83.33% <0%> (+1.19%) 0% <0%> (ø) ⬇️

@AndyScherzinger

Copy link
Copy Markdown
Member

@tobiasKaminsky I tried the latest QA build but it renders the "+" on the fab button in white (for dark) so I can see the icon (which seems to work according to your screenshots).

@tobiasKaminsky tobiasKaminsky added this to the Nextcloud App 3.10.0 milestone Jan 8, 2020
@tobiasKaminsky

Copy link
Copy Markdown
Member Author

@tobiasKaminsky I tried the latest QA build but it renders the "+" on the fab button in white (for dark) so I can see the icon (which seems to work according to your screenshots).

can you open up a new issue for this? Also which primary color do you use?

@AndyScherzinger

Copy link
Copy Markdown
Member

Done: #5110

tobiasKaminsky added a commit that referenced this pull request Jan 9, 2020
8751c85 Merge pull request #5120 from nextcloud/spinner
9d2b80d spinner: if primary color is white, use grey
de269b2 Merge pull request #5115 from nextcloud/311bump
3baa9fe Merge pull request #5118 from nextcloud/fastlane
ee6e7ce fix fastlane RC
1343ba0 Merge pull request #5114 from nextcloud/richWorkspaceFixes
1ec8b06 - fix not clickable rich workspace area - fix crash on first loading
62aaed7 bump to 3.11 alpha
9a47a4d Merge pull request #5107 from nextcloud/whiteHeader
6cb0f10 Merge pull request #4915 from nextcloud/createText
3011097 [tx-robot] updated from transifex
248fe1a daily dev 20200108
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.

Wrong setting header color after enabling dark

3 participants