-
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
[Android] Respect both image size and column width #4613
Conversation
app:barrierDirection="left" | ||
app:constraint_referenced_ids="widthPlaceholder" /> | ||
|
||
|
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.
nit: extra spaces
"type": "AdaptiveCard", |
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.
nice catch 👍
Last time there were problems with media and other elements, could you do a quick run across all cards to check that everything is working as expected? |
Test evidence added to PR description |
* 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 #4439
Description
Uses constraint layout to make images grow to the requested size, but not larger than the parent container/column.
How Verified
Manual verification:
v1.2/Tests/ImageSizing.json
:ActivityUpdate.json
:FlightItinerary.json
:ProductVideo.json
:Agenda.json
:Microsoft Reviewers: Open in CodeFlow