-
Notifications
You must be signed in to change notification settings - Fork 540
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
Revert "[Android] Use ConstraintLayout for Image sizing (#4461)" #4520
Conversation
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.
Thanks for this.
Reverting #4461 seems to fix the media issue when I test it (the one about casting from constraint params). What further issue did you see / what was the fix?
the other issue was horizontal alignment that is now fixed, but we're missing some testing |
@almedina-ms this gets CP'd into release/20.07 && release/1.2 as well yeah? |
What is the testing here and can we include the test cards in if they aren't already part of the initial fixes? |
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.
This needs to be CP'd over to release/1.2 and release/20.07 @almedina-ms thanks |
…)" (microsoft#4520) * Revert "[Android] Use ConstraintLayout for Image sizing (microsoft#4461)" This reverts commit 6fc552f. * Make media work again * Reset imageview gravity Co-authored-by: shalinijoshi19 <shalinij@microsoft.com>
* Revert "Revert "[Android] Use ConstraintLayout for Image sizing (microsoft#4461)" (microsoft#4520)" This reverts commit efb0e17. * Fix for containers with auto width * Fix media images * Switched to xml for clarity * Refactor, cleanup whitespace * Fix sample
Related Issue
Fixes #4518
Description
This reverts commit 805c51e.
Up to this point the media element still doesn't work, going back I was able to revert the break by going to commit 1202191 for ImageRenderer
How Verified
TBD
Microsoft Reviewers: Open in CodeFlow