-
Notifications
You must be signed in to change notification settings - Fork 517
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 #3564 Merge profile progress header xml #3885
Fixes #3564 Merge profile progress header xml #3885
Conversation
@rt4914 PTAL, as discussed I made a PR merging the 2 tab layout files but it says some conflicts I tried viewing them but got no idea how to resolve them and not understood the issue. |
@rt4914, I resolved the conflicts but one test is failing |
@bkaur-bkj As mentioned in the issue description please add before and after screenshot of mobile-portrait, mobile-landscape, tablet-portrait and tablet-landscape for comparison and make sure that there is not difference between before and after UI. |
@bkaur-bkj The test which is failing is an optional one and also it mostly is failing on develop too so don't worry about it in this PR. |
…ort to layout-sw600dp
@rt4914 I have added ss for mobile portrait and landscape but I tried for tablet it crashes, as I tried merging the landscape and portrait layout of tablet I removed tablet landscape layout and tried running at app on tab, it was fine with portrait but crashed in landscape so I changed the folder of the file and placed the file profile_progress_header in the layout/swa600dp/ folder. But it not crases for portrait layout too. |
@bkaur-bkj For tablet follow these steps:
|
@bkaur-bkj Also please merge with latest develop as there are conflicts currently. |
Yes sir Done |
yes done worked now |
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.
@bkaur-bkj 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.
One comment left.
@bkaur-bkj PTAL
@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.
LGTM, thanks.
Explanation
Fixes #3564
merged portrait and landscape files of tab layout, could not merge the mobile layout files as they had some constraints and other differences too.
Essential Checklist
For UI-specific PRs only
If your PR includes UI-related changes, then:
mobile landscape before
mobile landscape after changes
After changes tablet portrait
After changes tablet landscape