Skip to content
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] Refactor imageSize methods #4380

Merged
merged 4 commits into from
Jul 17, 2020
Merged

[Android] Refactor imageSize methods #4380

merged 4 commits into from
Jul 17, 2020

Conversation

golddove
Copy link
Member

@golddove golddove commented Jul 15, 2020

Related Issue

Fixes #4364, fixes #4043, fixes #4410 and various other image sizing inconsistencies

Description

Bring Android image sizing/defaults to match JS/iOS/spec.

How Verified

Manual verification.
Payload at samples/v1.2/Tests/ImageSizing.json.
Rendered card:
4364

Microsoft Reviewers: Open in CodeFlow

@golddove golddove merged commit 1202191 into main Jul 17, 2020
@golddove golddove deleted the golddove/4364 branch July 17, 2020 20:35
golddove added a commit that referenced this pull request Jul 31, 2020
* Refactor imageSize methods

* Add test card
golddove added a commit that referenced this pull request Jul 31, 2020
* Cherry-pick #4380

* Cherry-pick #4461

* Bad merge - duplicate call to setVisibility

Co-authored-by: almedina-ms <35784165+almedina-ms@users.noreply.github.com>
Co-authored-by: Paul Campbell <paulcam@microsoft.com>
rankush pushed a commit to rankush/AdaptiveCards that referenced this pull request May 8, 2024
* Refactor imageSize methods

* Add test card
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants