Skip to content

Dark theme: color enhancements#4762

Merged
tobiasKaminsky merged 1 commit into
masterfrom
color
Oct 30, 2019
Merged

Dark theme: color enhancements#4762
tobiasKaminsky merged 1 commit into
masterfrom
color

Conversation

@tobiasKaminsky

Copy link
Copy Markdown
Member
  • overwrite in values-night/colors only values that differ
  • move branding colors back to setup.xml

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

move branding colors back to setup.xml

Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
@AndyScherzinger

Copy link
Copy Markdown
Member

@tobiasKaminsky changes look good to me. Other than that a question: would you be fine with some renamings here? (I'd open an issue of course and then a PR; not sure when but looking through the colors there is some room for improvement regarding names naming pattern).

@tobiasKaminsky

Copy link
Copy Markdown
Member Author

Yeah, of course, just open up an issue :-)

@nextcloud-android-bot

Copy link
Copy Markdown
Collaborator

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

304

Lint

TypemasterPR
Warnings5959
Errors00

SpotBugs (new)

Warning TypeNumber
Bad practice Warnings26
Correctness Warnings69
Internationalization Warnings13
Malicious code vulnerability Warnings5
Multithreaded correctness Warnings9
Performance Warnings119
Security Warnings44
Dodgy code Warnings138
Total423

SpotBugs (master)

Warning TypeNumber
Bad practice Warnings26
Correctness Warnings69
Internationalization Warnings13
Malicious code vulnerability Warnings5
Multithreaded correctness Warnings9
Performance Warnings119
Security Warnings44
Dodgy code Warnings138
Total423

@codecov

codecov Bot commented Oct 30, 2019

Copy link
Copy Markdown

Codecov Report

Merging #4762 into master will decrease coverage by 0.49%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             master    #4762     +/-   ##
===========================================
- Coverage     17.97%   17.48%   -0.5%     
  Complexity        3        3             
===========================================
  Files           376      376             
  Lines         32383    32383             
  Branches       4568     4568             
===========================================
- Hits           5822     5663    -159     
- Misses        25596    25801    +205     
+ Partials        965      919     -46
Impacted Files Coverage Δ Complexity Δ
...ava/com/owncloud/android/files/FileMenuFilter.java 0% <0%> (-52.06%) 0% <0%> (ø)
.../third_parties/daveKoeller/AlphanumComparator.java 79.76% <0%> (-3.58%) 0% <0%> (ø)
...owncloud/android/ui/adapter/OCFileListAdapter.java 29% <0%> (-3.06%) 0% <0%> (ø)
...ncloud/android/ui/fragment/OCFileListFragment.java 25.3% <0%> (-2.44%) 0% <0%> (ø)
...in/java/com/owncloud/android/utils/ThemeUtils.java 50% <0%> (-1.58%) 0% <0%> (ø)
.../java/com/owncloud/android/utils/DisplayUtils.java 23.62% <0%> (-1.27%) 0% <0%> (ø)
...loud/android/datamodel/FileDataStorageManager.java 36.54% <0%> (-0.72%) 0% <0%> (ø)
...loud/android/datamodel/ThumbnailsCacheManager.java 35.58% <0%> (-0.19%) 0% <0%> (ø)
...oud/android/ui/activity/SyncedFoldersActivity.java 22.79% <0%> (ø) 0% <0%> (ø) ⬇️
...xtcloud/client/account/UserAccountManagerImpl.java 50% <0%> (ø) 0% <0%> (ø) ⬇️
... and 3 more

@tobiasKaminsky
tobiasKaminsky merged commit c4e20a6 into master Oct 30, 2019
@delete-merged-branch
delete-merged-branch Bot deleted the color branch October 30, 2019 10:08
@AndyScherzinger AndyScherzinger added this to the Nextcloud App 3.10.0 milestone Oct 30, 2019
tobiasKaminsky added a commit that referenced this pull request Oct 31, 2019
c4e20a6 Merge pull request #4762 from nextcloud/color
af63eed overwrite in values-night/colors only values that differ move branding colors back to setup.xml
a9fcc7b daily dev 20191030
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