-
Notifications
You must be signed in to change notification settings - Fork 499
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
Fix #3461 : Merged App Version fragment and app version activity into individual files #3683
Fix #3461 : Merged App Version fragment and app version activity into individual files #3683
Conversation
@anandwana001 @rt4914 PTAL |
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.
Left some comments, PTAL @Rohit1173
@prayutsu PTAL |
Unassigning @Rohit1173 since a re-review was requested. @Rohit1173, please make sure you have addressed all review comments. Thanks! |
@Rohit1173 You should never mark conversation resolved if you didn't initiate the comment. They should only be resolved by the reviewers. |
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.
LGTM, Thanks @Rohit1173
I will make sure this doesn't happen again |
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.
@Rohit1173 Please take the values again it looks like the values have been used incorrectly.
@rt4914 PTAL |
Unassigning @Rohit1173 since a re-review was requested. @Rohit1173, please make sure you have addressed all review comments. Thanks! |
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.
@Rohit1173 PTAL
@rt4914 PTAL |
Unassigning @Rohit1173 since a re-review was requested. @Rohit1173, please make sure you have addressed all review comments. Thanks! |
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.
LGTM, thanks.
Explanation
Fix #3461
There were 3 files of app_version_fragment, I have and I compared them in the given website and added the specific changes in their respective dimens.xml files.
There were 2 files of app_version_activity, and both were identical so I have deleted the Landscape file
Here are the screenshots of the app before and after the above-mentioned changes
Checklist