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

improve UI of accounts screen #115

Open laxyapahuja opened 4 years ago

laxyapahuja commented 4 years ago

Issue Fix

Fixes #103

Description

Updated the UI to the new redesign design.

Screenshots

rchtgpt commented 4 years ago

hey @laxyapahuja, you'll have to add all the dimensions, texts and colors in dimens.xml, strings.xml and colors.xml respectively.

laxyapahuja commented 4 years ago

updated.

rchtgpt commented 4 years ago

I can still see some hardcoded texts, colors and dimensions. can you check again?

laxyapahuja commented 4 years ago

Updated.

laxyapahuja commented 4 years ago

@rachittt all this code was already existing. i haven't touched it and it is not advised to correct already existing code :)

rchtgpt commented 4 years ago

@laxyapahuja, when the user clicks on the loan button, a plus icon will appear on the top right side which will redirect the user to Apply for Loan screen. Please check the prototype video of mockup design from '0:15' to get reference.

laxyapahuja commented 4 years ago

@laxyapahuja, when the user clicks on the loan button, a plus icon will appear on the top right side which will redirect the user to Apply for Loan screen. Please check the prototype video of mockup design from '0:15' to get reference.

updated.

luckyman20 commented 4 years ago

Use RecyclerView to show lists.

laxyapahuja commented 4 years ago

Use RecyclerView to show lists.

There already was a recycler view for recent transactions. I've used that fragment as a nested fragment in dashboard activity as that was the most optimal solution and didn't require me to change the code much.

garvit984 commented 4 years ago

The tint color for deposit and loan is not correct. Cards are also not as per the design. The position, size, and color of the plus symbol are not correct. Remove that Toolbar.

garvit984 commented 4 years ago

@laxyapahuja Please correct the conflicting files and the changes requested above.