Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes #1562 UI improvements in client details page #1563

Merged

Conversation

danishjamal104
Copy link
Collaborator

Fixes #1562

Changes

  • Added drawable round_table_row_bg.xml for rounded corners in a table row.
  • Center aligned the image.
  • Client name is now below the image instead of to the right.
  • Added Accounts header to separate the client info table and accounts elements.
  • Modified the padding and margin of account_accordion section

Screenshot

Screenshot 2020-11-19 at 5 10 13 PM

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

  • Apply the MifosStyle.xml style template to your code in Android Studio.

  • Run the unit tests with ./gradlew check to make sure you didn't break anything

  • If you have multiple commits please combine them into one commit by squashing them.

@danishjamal104 danishjamal104 force-pushed the ui-improvements-client-info branch from 209ed33 to caefc4a Compare November 20, 2020 05:55
Copy link
Contributor

@PatelVatsalB21 PatelVatsalB21 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@iamsh4shank iamsh4shank left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you fix the merge conflict so that we can merge it:)

@danishjamal104
Copy link
Collaborator Author

Can you fix the merge conflict so that we can merge it:)

done with the changes, please check.

Copy link
Member

@iamsh4shank iamsh4shank left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Squash your commits as well

@danishjamal104 danishjamal104 force-pushed the ui-improvements-client-info branch 2 times, most recently from caefc4a to 415bb6b Compare June 3, 2021 15:30
@danishjamal104
Copy link
Collaborator Author

@iamsh4shank kindly merge this. Thanks :)

Copy link
Member

@iamsh4shank iamsh4shank left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@iamsh4shank
Copy link
Member

@danishjamal104 can you fix the build fail so that I will merge it asap

@danishjamal104 danishjamal104 force-pushed the ui-improvements-client-info branch 3 times, most recently from a879031 to 9304ba0 Compare June 23, 2021 09:29
@danishjamal104 danishjamal104 force-pushed the ui-improvements-client-info branch from 9304ba0 to bcfdc5a Compare June 23, 2021 09:31
@danishjamal104
Copy link
Collaborator Author

danishjamal104 commented Jun 23, 2021

The build is passing successfully on my local machine but failing on the travis, see below two screen shot @iamsh4shank. Thanks :)

Travis Screenshot of error

Screenshot 2021-06-23 at 3 12 46 PM

Local build screen shot of task:mifosng-android:pmd

Screenshot 2021-06-23 at 3 13 05 PM

@iamsh4shank iamsh4shank changed the base branch from master to release-1.1 June 23, 2021 15:13
Copy link
Member

@iamsh4shank iamsh4shank left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah that's fine

@iamsh4shank iamsh4shank merged commit 71d3d41 into openMF:release-1.1 Jun 23, 2021
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.

Enhancement: Improvement of UI of client info in fragment_client_details.xml
3 participants