Skip to content

Conversation

@AndyScherzinger
Copy link
Member

@AndyScherzinger AndyScherzinger commented Nov 5, 2018

Move to new Android X implementation and thus upgrade to latest material design theming.

DONE

  • Use sentence case for button labels
  • Use primary color for status bar too (instead of primary dark color), like refresh Google apps do too
  • replace accent color with primary color
  • use new outlined style for secondary buttons and for textfields, use pill style buttons.

Screenshot on Android 9
device-2018-12-28-152236

device-2018-12-28-152357

@AndyScherzinger AndyScherzinger changed the base branch from master to androidX November 5, 2018 21:19
@AndyScherzinger AndyScherzinger force-pushed the androidX branch 3 times, most recently from 4cb0f3f to 925383c Compare December 3, 2018 16:18
@tobiasKaminsky tobiasKaminsky force-pushed the androidX branch 3 times, most recently from 0be5d9c to 59cf6b9 Compare December 10, 2018 12:11
@AndyScherzinger AndyScherzinger changed the base branch from androidX to master December 10, 2018 23:20
@AndyScherzinger AndyScherzinger force-pushed the materialDesignRevamp branch 2 times, most recently from 0d4a386 to 9858cc7 Compare December 12, 2018 12:52
@tobiasKaminsky
Copy link
Member

Where is this crashing? Then i can try to fix it.
A first smoke test did not lead to major crashes.

@AndyScherzinger
Copy link
Member Author

@tobiasKaminsky I did an additional commit which introduced the usage of the bridge theme which prevents the crashes and allow for mixed UI elements ;)

@tobiasKaminsky
Copy link
Member

Rebased!

@AndyScherzinger
Copy link
Member Author

@jancborchardt please see the comment by @tobiasKaminsky regarding the buttons on the login screen. ❤️

@github-tomster
Copy link

hey a revamp, how about a theme switch?
light/dark and maybe AMOLED compatible BLACK theme?
saves battery (the black amoled only) and this brings a better user experience.
if there is no way to make a black theme maybe only a dark theme at last.
so the user can switch theme in this application too. like in many apps around the www.
see:
https://github.com/stefan-niedermann/nextcloud-notes
there is an theme switch. users like it.

@AndyScherzinger
Copy link
Member Author

Hi @github-tomster thanks for the feedback. Dark theme is out of scope for this PR but is in discussion in another issue. The short version: it's on our radar and will come in the future, the long version: it'll take a while to be done since it is a bit more complicated than with the mentioned app. This client (files app) already implemented dynamic theming based on the server side theme so we do calculate some colors already based on a base/primary color from the server and for a dark theme these calculations will have to work a bit different plus with the server probably having proper dark theme support in the future we will have to support that too in a proper way.

So again: it'll come but might take a while due to limited time. If you or anybody else is willing to step up and implement it, please, feel free and be sure we will support you wherever we can to get it done.

If nobody steps up I'll probably implement it but won't have the time to start working on it before February or March... Busy day job...

@jancborchardt
Copy link
Member

@AndyScherzinger @tobiasKaminsky right, the primary button needs to be a bit adjusted to work on the blue (same-colored) background. Can we use a border? Quick mockup based on the web interface log in screen:
log in and sign up with provider

@github-tomster the dark theme is tracked at #2188 and being worked on by awesome community member @dan0xii. :)

@AndyScherzinger
Copy link
Member Author

AndyScherzinger commented Jan 8, 2019

Sure thing... need to have to separate style implemented here since the second button would be an outlined one then (like the new secondary action buttons) 👍
Arrow is probably not on the button, but maybe possible.

@jancborchardt
Copy link
Member

Yeah, nevermind the arrow – it’s just how it is on the server. :)

AndyScherzinger and others added 12 commits January 12, 2019 15:27
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
@AndyScherzinger AndyScherzinger force-pushed the materialDesignRevamp branch 2 times, most recently from 803e6c0 to 898f9b9 Compare January 12, 2019 15:02
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
@nextcloud-android-bot
Copy link
Collaborator

Lint

TypemasterPR
Warnings8383
Errors00

FindBugs (new)

Warning TypeNumber
Bad practice Warnings35
Correctness Warnings157
Internationalization Warnings15
Malicious code vulnerability Warnings12
Multithreaded correctness Warnings9
Performance Warnings118
Security Warnings66
Dodgy code Warnings135
Total547

FindBugs (master)

Warning TypeNumber
Bad practice Warnings35
Correctness Warnings158
Internationalization Warnings15
Malicious code vulnerability Warnings12
Multithreaded correctness Warnings9
Performance Warnings118
Security Warnings66
Dodgy code Warnings135
Total548

@AndyScherzinger AndyScherzinger merged commit a201dae into master Jan 12, 2019
@AndyScherzinger AndyScherzinger added this to the Nextcloud App 3.5.0 milestone Jan 12, 2019
@tobiasKaminsky tobiasKaminsky deleted the materialDesignRevamp branch January 14, 2019 06:35
@tobiasKaminsky
Copy link
Member

🎉 cool to have this in

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.

6 participants