openMF / mifos-mobile-cn

Client-facing app on top of Apache Fineract CN
https://openmf.github.io/mobileapps.github.io/
Apache License 2.0
38 stars 64 forks source link

redesign: Improve UI of Login Screen #162

Closed prajwal27 closed 4 years ago

prajwal27 commented 4 years ago

Fixes #100

Summary Improve the UI of Login Screen. Also, virtual keyboard to hide whenever clicked on the screen.

Expected Behaviour ezgif com-video-to-gif (3)

Please make sure these boxes are checked before submitting your pull request - thanks!

jawidMuhammadi commented 4 years ago

Hey, @prajwal27 Your designed UI looks fantastic. But please keep in mind that whenever we design or redesign any screen we should keep the entire App's design consistent which is very important from material design and App's branding perspectives. For example, we have not used Outlined Edit Text in any other place inside the app and the background color does not have any relation with colorPrimary, colorPrimaryDark and colorAccent. So I recommend you to use Edit Texts without outline style and for background color use our predefined colors or something new but very similar to them.

prajwal27 commented 4 years ago

Hey, @prajwal27 Your designed UI looks fantastic. But please keep in mind that whenever we design or redesign any screen we should keep the entire App's design consistent which is very important from material design and App's branding perspectives. For example, we have not used Outlined Edit Text in any other place inside the app and the background color does not have any relation with colorPrimary, colorPrimaryDark and colorAccent. So I recommend you to use Edit Texts without outline style and for background color use our predefined colors or something new but very similar to them.

@jawidMuhammadi since I had hoped to re-design only the login screen, theses inconsistencies are present.

devansh-299 commented 4 years ago

@prajwal27 This task is meant to be performed during the GSoC period. I don't think you should work on it now

rchtgpt commented 4 years ago

@prajwal27, this PR is a duplicate of #110. We cannot consider this PR until #110 has been reviewed and closed therefore, closing this PR for now. We do appreciate your efforts though