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

Rounding off to 2 decimals for constraint multiplier for Image Renderer #6863

Conversation

shabripabba
Copy link
Contributor

@shabripabba shabripabba commented Jan 3, 2022

Related Issue

Fixed #6846
Teams application when using a subclass of ACRImageRenderer as a CustomImageRenderer, crashes on some magic ratio number, when a card of a certain payload is expanded.

Description

Issue is being fixed by making sure that ratio is rounded of to 2 decimals, keeping the image as accurate as it can be, and avoiding a magic ratio number on which apple has some crash

Sample Card

{
"type": "AdaptiveCard",
"version": "1.0",
"body": [
{
"height": "auto",
"type": "Image",
"url": "https://picsum.photos/744/84",
"altText": "Power Automate approvals logo"
},
{
"size": "large",
"weight": "bolder",
"type": "TextBlock",
"text": "Review new SharePoint file 'AppInstallViewData.json'"
},
{
"type": "TextBlock",
"text": "Requested by Sagar Pant Pant.Sagar@microsoft.com",
"wrap": true
},
{
"facts": [
{
"title": "Date Created",
"value": "{{DATE(2021-11-26T11:50:05Z, LONG)}} {{TIME(2021-11-26T11:50:05Z)}}"
},
{
"title": "Link",
"value": "AppInstallViewData.json"
}
],
"type": "FactSet"
},
{
"wrap": true,
"id": "ApprovalDetailsSection",
"type": "TextBlock",
"text": "- Added to: Shared Documents/\n- Created by: Sagar Pant Pant.Sagar@microsoft.com\n- Created time (UTC): 2021-11-26T11:49:29Z"
},
{
"actions": [
{
"title": "Approve",
"card": {
"body": [
{
"type": "TextBlock",
"text": "Comments",
"wrap": true
},
{
"id": "comments1",
"placeholder": "Enter comments",
"type": "Input.Text",
"isMultiline": true,
"maxLength": 2048
}
],
"type": "AdaptiveCard",
"actions": [
{
"title": "Submit",
"data": {
"OnBehalfOfNotice": "Requested by Sagar Pant Pant.Sagar@microsoft.com",
"ApprovalName": "53243a00-68e8-42c0-87bd-5964e9d322c9",
"SelectedOption": "Approve",
"Options": [
"Approve",
"Reject"
],
"MessageTitle": "Review new SharePoint file 'AppInstallViewData.json'",
"ApprovalTitle": "Review new SharePoint file 'AppInstallViewData.json'",
"ApprovalLink": "https://flow.microsoft.com/manage/environments/839eace6-59ab-4243-97ec-a5b8fcc104e4/approvals/received/53243a00-68e8-42c0-87bd-5964e9d322c9?loginTenant=72f988bf-86f1-41af-91ab-2d7cd011db47",
"ItemLinkDescription": "AppInstallViewData.json",
"Details": "- Added to: Shared Documents/\n- Created by: Sagar Pant Pant.Sagar@microsoft.com\n- Created time (UTC): 2021-11-26T11:49:29Z",
"Environment": "839eace6-59ab-4243-97ec-a5b8fcc104e4",
"CreatorName": "Sagar Pant",
"CreationTime": ""Friday, 26 November 2021 11:50 GMT"",
"ActionType": 1,
"CreatorEmail": "Pant.Sagar@microsoft.com",
"ItemLink": "https://microsoftapc.sharepoint.com/teams/Testteam2182/Shared%20Documents/AppInstallViewData.json"
},
"type": "Action.Submit"
}
],
"version": "1.0"
},
"type": "Action.ShowCard"
},
{
"title": "Reject",
"card": {
"body": [
{
"type": "TextBlock",
"text": "Comments",
"wrap": true
},
{
"id": "comments2",
"placeholder": "Enter comments",
"type": "Input.Text",
"isMultiline": true,
"maxLength": 2048
}
],
"type": "AdaptiveCard",
"actions": [
{
"title": "Submit",
"data": {
"OnBehalfOfNotice": "Requested by Sagar Pant Pant.Sagar@microsoft.com",
"ApprovalName": "53243a00-68e8-42c0-87bd-5964e9d322c9",
"SelectedOption": "Reject",
"Options": [
"Approve",
"Reject"
],
"MessageTitle": "Review new SharePoint file 'AppInstallViewData.json'",
"ApprovalTitle": "Review new SharePoint file 'AppInstallViewData.json'",
"ApprovalLink": "https://flow.microsoft.com/manage/environments/839eace6-59ab-4243-97ec-a5b8fcc104e4/approvals/received/53243a00-68e8-42c0-87bd-5964e9d322c9?loginTenant=72f988bf-86f1-41af-91ab-2d7cd011db47",
"ItemLinkDescription": "AppInstallViewData.json",
"Details": "- Added to: Shared Documents/\n- Created by: Sagar Pant Pant.Sagar@microsoft.com\n- Created time (UTC): 2021-11-26T11:49:29Z",
"Environment": "839eace6-59ab-4243-97ec-a5b8fcc104e4",
"CreatorName": "Sagar Pant",
"CreationTime": ""Friday, 26 November 2021 11:50 GMT"",
"ActionType": 1,
"CreatorEmail": "Pant.Sagar@microsoft.com",
"ItemLink": "https://microsoftapc.sharepoint.com/teams/Testteam2182/Shared%20Documents/AppInstallViewData.json"
},
"type": "Action.Submit"
}
],
"version": "1.0"
},
"type": "Action.ShowCard"
}
],
"type": "ActionSet"
}
]
}

How Verified

Verified the fix using manual testing

  1. tested whether the test app is still running and the above card is displayed properly and that "Approve" button works on the test app.
  2. Verified this change and added it manually in client (teams) app to check if the crash would be fixed.
Microsoft Reviewers: Open in CodeFlow

@jwoo-msft jwoo-msft enabled auto-merge (squash) January 3, 2022 23:34
@jwoo-msft jwoo-msft merged commit 0e505ff into microsoft:main Jan 4, 2022
jwoo-msft pushed a commit that referenced this pull request Jan 6, 2022
paulcam206 pushed a commit that referenced this pull request Jan 6, 2022
* Added App extension to podspec (#6849)

* app extension model added

updated spec

added podspec at root for testing\n revert it

spec updated

updated pod spec

pod spec update

moretesting

[updated macro name]

* updated podspec

* [ios] Updated to final podspec version

* update compiler flag

* pod spec update

* updated flag name

* [iOS] fixed empty filtered list issue (#6528)

* [iOS] fixed empty filtered list issue

* [iOS] changed to function call

Co-authored-by: Canhua Li <licanhua@live.com>

* moving resource bundle to core (#6853)

Co-authored-by: Janus Balatbat <jsbalatbat@microsoft.com>

* Rounding off to 2 decimals (#6863)

* [ios] updated round off logic (#6866)

* [ios] updated round off logic

* [ios] refactored the code

* [ios] fixed project file

* [ios] fixed log

* [ios] spec updated

* update test (#6864)

* added heartbeat test ios build infrastructure (#6842)

* [iOS] added yml version of ci

updated path

[ios] path update

[iOS] variable updated

[ios] added new test scheme

* [iOS] added spec linting task

added missing test file

updated spect lint task

added error handling

added parallelism to jobs

* added cert test

added cert install yml

updated pass

Update ios-heartbeat.yml for Azure Pipelines

added provisional file

added script

unimportant commit

unimportant commit

unimportant commit

* added e-mail message

simple update

added e-mail message

added e-mail message

added e-mail message

updated e-mail message

updated e-mail message

updated e-mail message

* updated cron job

* added false test

Revert "added false test"

This reverts commit ef302f0.

* added false test

added build task to cert task

added build task to cert task

added build task to cert task

good cert bad prov

added app push

* parameterized template

updated parameters

updated params

updated parameters

updated to correct provisional file

* updated trigger

* updated e-mail address

Co-authored-by: Canhua Li <licanhua@live.com>
Co-authored-by: janusbal <86079148+janusbal@users.noreply.github.com>
Co-authored-by: Janus Balatbat <jsbalatbat@microsoft.com>
Co-authored-by: shabripabba <87400370+shabripabba@users.noreply.github.com>
jwoo-msft added a commit that referenced this pull request Feb 17, 2022
* Added App extension to podspec (#6849)

* app extension model added

updated spec

added podspec at root for testing\n revert it

spec updated

updated pod spec

pod spec update

moretesting

[updated macro name]

* updated podspec

* [ios] Updated to final podspec version

* update compiler flag

* pod spec update

* updated flag name

* [iOS] fixed empty filtered list issue (#6528)

* [iOS] fixed empty filtered list issue

* [iOS] changed to function call

Co-authored-by: Canhua Li <licanhua@live.com>

* moving resource bundle to core (#6853)

Co-authored-by: Janus Balatbat <jsbalatbat@microsoft.com>

* Rounding off to 2 decimals (#6863)

* [ios] updated round off logic (#6866)

* [ios] updated round off logic

* [ios] refactored the code

* [ios] fixed project file

* [ios] fixed log

* [ios] spec updated

* update test (#6864)

* added heartbeat test ios build infrastructure (#6842)

* [iOS] added yml version of ci

updated path

[ios] path update

[iOS] variable updated

[ios] added new test scheme

* [iOS] added spec linting task

added missing test file

updated spect lint task

added error handling

added parallelism to jobs

* added cert test

added cert install yml

updated pass

Update ios-heartbeat.yml for Azure Pipelines

added provisional file

added script

unimportant commit

unimportant commit

unimportant commit

* added e-mail message

simple update

added e-mail message

added e-mail message

added e-mail message

updated e-mail message

updated e-mail message

updated e-mail message

* updated cron job

* added false test

Revert "added false test"

This reverts commit ef302f0.

* added false test

added build task to cert task

added build task to cert task

added build task to cert task

good cert bad prov

added app push

* parameterized template

updated parameters

updated params

updated parameters

updated to correct provisional file

* updated trigger

* updated e-mail address

* Bug fixes: Textblock layout | Parsing warnings (#7041)

* need to take accoutn for empty

* works but need to find memory leak

* testing against stackedit

* removing unneeded line | allocating array

* clean up

* getting rid of str() use from strigstream

* handling link destination with [ character and other tokens

* removing cpp changes

Co-authored-by: Janus Balatbat <jsbalatbat@microsoft.com>

* Bug fixes: Handling of link markdown  (#6922)

* need to take accoutn for empty

* works but need to find memory leak

* testing against stackedit

* removing unneeded line | allocating array

* clean up

* getting rid of str() use from strigstream

* handling link destination with [ character and other tokens

* removing the ios changes

* resolving comments

* resolving comments

* clan format

Co-authored-by: Janus Balatbat <jsbalatbat@microsoft.com>

* updated android-ci yml to use latest macOs

Co-authored-by: Canhua Li <licanhua@live.com>
Co-authored-by: janusbal <86079148+janusbal@users.noreply.github.com>
Co-authored-by: Janus Balatbat <jsbalatbat@microsoft.com>
Co-authored-by: shabripabba <87400370+shabripabba@users.noreply.github.com>
michaelfarnsworth pushed a commit to michaelfarnsworth/AdaptiveCards that referenced this pull request Nov 10, 2022
rankush pushed a commit to rankush/AdaptiveCards that referenced this pull request May 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Rendering] Crash on teams iOS platform, due to conflicting constraints
3 participants