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.

Sign up for GitHub

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

Jump to bottom

Image takes more space than needed if RequiredWidth is not set #5438

Closed
Dreamescaper opened this issue Mar 19, 2022 · 2 comments
Closed

Image takes more space than needed if RequiredWidth is not set #5438

Dreamescaper opened this issue Mar 19, 2022 · 2 comments
Labels
area/controls 🎮 Label, Button, CheckBox, Slider, Stepper, Switch, Picker, Entry, Editor p/2 Work that is important, but not critical for the release platform/android 🤖 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Projects
Triage
Milestone
6.0-servicing

Comments

@Dreamescaper
Copy link

Description

If RequiredHeight is set, and RequieredWidth is not set, image takes more space than needed.

It reproduces on Android, but works fine on Windows, and I wasn't able to test it on iOS/MacOS.

Steps to Reproduce

Use the following page content:

    <HorizontalStackLayout>
        <Label Text="Icon: "/>

        <Image HeightRequest="20" 
                     Source="https://cdn4.iconfinder.com/data/icons/web-ui-color/128/Compose-512.png" 
                     BackgroundColor="LightBlue"/>

        <Label Text=" edit"/>
    </HorizontalStackLayout>

Expected:
image

Actual:
image

Version with bug

Preview 14 (current)

Last version that worked well

Unknown/Other

Affected platforms

Android

Affected platform versions

Android 12

Did you find any workaround?

Set RequiredWidth.

Relevant log output

No response
@Dreamescaper Dreamescaper added s/needs-verification Indicates that this issue needs initial verification before further triage will happen t/bug Something isn't working labels Mar 19, 2022
@XamlTest XamlTest added the s/triaged Issue has been reviewed label Mar 21, 2022
@XamlTest
Copy link
Collaborator

Verified repro with Visual Studio Enterprise 2022 main 17.2.0 Preview 2.0 [32223.438.main] on Android 12. Repro Project: 5438.zip

@XamlTest XamlTest added platform/android 🤖 s/verified Verified / Reproducible Issue ready for Engineering Triage and removed s/needs-verification Indicates that this issue needs initial verification before further triage will happen labels Mar 21, 2022
@jsuarezruiz jsuarezruiz added this to Ready For Work in Triage Mar 21, 2022
@Redth Redth added this to the 6.0.300 milestone Mar 21, 2022
@Eilon Eilon added the area/controls 🎮 Label, Button, CheckBox, Slider, Stepper, Switch, Picker, Entry, Editor label Mar 28, 2022
@Dreamescaper Dreamescaper mentioned this issue Mar 28, 2022
Closed
@davidortinau davidortinau added the p/1 Work that is critical for the release, but we could probably ship without label Apr 29, 2022
@Redth Redth added p/2 Work that is important, but not critical for the release and removed p/1 Work that is critical for the release, but we could probably ship without labels May 3, 2022
@samhouts samhouts modified the milestones: 6.0.300, 6.0.300-servicing May 4, 2022
@Dreamescaper Dreamescaper mentioned this issue May 21, 2022
Closed
@hartez
Copy link
Contributor

hartez commented May 23, 2022

Actually, #7389 fixes this.

@hartez hartez closed this as completed May 23, 2022
Triage automation moved this from Ready For Work to Done May 23, 2022
@dotnet dotnet locked as resolved and limited conversation to collaborators Jun 22, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
area/controls 🎮 Label, Button, CheckBox, Slider, Stepper, Switch, Picker, Entry, Editor p/2 Work that is important, but not critical for the release platform/android 🤖 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Projects
No open projects
Triage
Done
Milestone
6.0-servicing
Development

No branches or pull requests
7 participants
@davidortinau @Eilon @Redth @hartez @samhouts @Dreamescaper @XamlTest