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] Background image squished. #4687

Closed
Gunashree-R-V opened this issue Aug 28, 2020 · 7 comments · Fixed by #4956
Closed

[Android] Background image squished. #4687

Gunashree-R-V opened this issue Aug 28, 2020 · 7 comments · Fixed by #4956
Assignees
Labels
Bug High Priority Partner-MsftTeams Issues related to AC integration with Microsoft Teams partner Partner-MsftTeamsMobile Bugs related specifically to Teams Mobile integration (iOS/Android) Platform-Android
Milestone

Comments

@Gunashree-R-V
Copy link

Platform

  • Android

Author or host

Microsoft Teams

Version of SDK

1.2.10

Details

Here is the JSON:

When a background image has fillMode: "repeat"/"repeatHorizontally"/"repeatVertically", the image is compressed.
And this happens irrespective of where the background image is placed.

Here is the Desktop, Android and iOS comparison:

Desktop Android iOS
image image image
{
    "type": "AdaptiveCard",
    "body": [
        {
            "type": "Container",
            "minHeight": "150px",
            "backgroundImage": "https://adaptivecards.io/content/AlkiBeach.jpg",
            "items": [
                {
                    "type": "TextBlock",
                    "text": "What a beautiful background"
                }
            ]
        },
        {
            "type": "TextBlock",
            "text": "They can even repeat a bunch of different ways..."
        },
        {
            "type": "Container",
            "minHeight": "100px",
            "backgroundImage": {
                "url": "https://adaptivecards.io/content/uparrow.png",
                "fillMode": "Repeat"
            }
        },
        {
            "type": "Container",
            "backgroundImage": {
                "url": "https://adaptivecards.io/content/uparrow.png",
                "fillMode": "RepeatHorizontally",
                "verticalAlignment": "Center"
            }
        },
        {
            "type": "Container",
            "minHeight": "100px",
            "backgroundImage": {
                "url": "https://adaptivecards.io/content/uparrow.png",
                "fillMode": "RepeatVertically",
                "horizontalAlignment": "Center"
            }
        }
    ],
    "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
    "version": "1.2"
}

cc: @VikrantSingh01, @anbah34, @shbhadra

@ghost ghost added the Triage-Needed label Aug 28, 2020
@ghost ghost removed the Triage-Needed label Aug 31, 2020
@ghost
Copy link

ghost commented Aug 31, 2020

Hi @Gunashree-R-V. We have acknowledged this issue report. Please continue to follow this issue for updates/progress/questions.

@ghost ghost added the Triage-Investigate label Aug 31, 2020
@golddove golddove self-assigned this Sep 9, 2020
@shalinijoshi19 shalinijoshi19 added Partner-MsftTeamsMobile Bugs related specifically to Teams Mobile integration (iOS/Android) Partner-MsftTeams Issues related to AC integration with Microsoft Teams partner Platform-Android Priority-Normal labels Sep 10, 2020
@shalinijoshi19
Copy link
Member

@golddove please update with your investigation comments here. Moving to 20.10 based on priorities around Accessibility for 9/30

@shalinijoshi19 shalinijoshi19 added this to the 20.10 milestone Sep 10, 2020
@golddove
Copy link
Member

Talked with @dclaux about this before - the same issue occurs with Image that has size: auto. Android shows a much smaller image than JS.

This needs a design discussion about how we want to treat images when there's no explicit width/stretch given.

Our spec just says that auto means no scaling (as long as it fits). This would mean every device would show a different sized image subject to the display's pizel density. But perhaps we want to instead scale them such that the same images take up the same physical size across devices..?

@shalinijoshi19
Copy link
Member

@golddove currently working on this. Testing out/Validating fix.

@ghost ghost added the AdaptiveCards v20.10 label Oct 22, 2020
@shalinijoshi19
Copy link
Member

@golddove updates? Is this still on track to be checked in by EOW?

@golddove
Copy link
Member

@golddove updates? Is this still on track to be checked in by EOW?

Should be good. In PR.

@ghost ghost added the Status-Fixed label Oct 27, 2020
@ghost
Copy link

ghost commented Oct 31, 2020

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug High Priority Partner-MsftTeams Issues related to AC integration with Microsoft Teams partner Partner-MsftTeamsMobile Bugs related specifically to Teams Mobile integration (iOS/Android) Platform-Android
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants