-
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 #3532: Deleted help_activity.xml (land) file #3597
Fix #3532: Deleted help_activity.xml (land) file #3597
Conversation
@Noobmaster-UD The |
@rt4914 I have uploaded the |
Unassigning @Noobmaster-UD since a re-review was requested. @Noobmaster-UD, please make sure you have addressed all review comments. Thanks! |
@UttaranDas The drawer screenshots are incorrect. The drawer should be opened in |
I have now updated the PR with the correct screenshots. |
@UttaranDas Do one thing, in This will make UI change but this is correct. This was you will need to keep only one file and the UI output will be much pleasant. |
…yout_width with it
I made the changes. Now the drawer looks like this:
@rt4914 PTAL |
Unassigning @UttaranDas since a re-review was requested. @UttaranDas, 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.
One nit change suggested. Thanks.
Hi @UttaranDas, I'm going to mark this PR as stale because it hasn't had any updates for 7 days. If no further activity occurs within 3 days, it will be automatically closed so that others can take up the issue. |
Explanation
Fix #3532
I have deleted the help_activity.xml (land) file. I compared the two xml files and found just one difference which was the android:layout_width. It was set to match_parent in the normal file whereas it was set to 304dp in land xml file. Deleting the land xml file changed the width of the drawer for the landscape mode. The app works stable so far. Following are the screenshots attached:
Drawer Screenshots
Checklist