-
Notifications
You must be signed in to change notification settings - Fork 1.3k
resolved issue #2542 with position of map icon #2601
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
resolved issue #2542 with position of map icon #2601
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2601 +/- ##
=========================================
- Coverage 2.7% 2.69% -0.01%
=========================================
Files 258 258
Lines 12283 12311 +28
Branches 1113 1116 +3
=========================================
Hits 332 332
- Misses 11925 11953 +28
Partials 26 26
Continue to review full report at Codecov.
|
@maskaravivek @nicolas-raoul @ujjwalagrawal17 The PR is complete and up for review. Can you please review :) |
@BindView(R.id.right_card) CardView rightCard; | ||
@BindView(R.id.right_card_expand_button) ImageView rightCardExpandButton; | ||
|
||
//@BindView(R.id.right_card_expand_button) ImageView rightCardExpandButton; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please can you remove the commented code - if we do need it later we can always go back into version control.
@vanshikaarora Can you attach a screenshot? |
@vanshikaarora Can you resolve the merge conflicts? |
Sure @maskaravivek :) I just made the changes and the PR is ready for review :) |
@maskaravivek I have resolved the merge conflicts Kindly review :) |
@maskaravivek here is a screenshot for the current position of map icon |
Thanks for approving the PR @maskaravivek :) |
Removed MapIcon from right card and added it to the bottom card
Fixes #2598 Put map component in bottom sheet of upload activity
Changes made