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

Label LineHeight causes bad layout under iOS #7222

Open
philipag opened this issue May 16, 2022 · 3 comments
Open

Label LineHeight causes bad layout under iOS #7222

philipag opened this issue May 16, 2022 · 3 comments
Labels
area-controls-label Label, Span area-layout StackLayout, GridLayout, ContentView, AbsoluteLayout, FlexLayout, ContentPresenter platform/iOS 🍎 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Milestone

Comments

@philipag
Copy link

philipag commented May 16, 2022

Description

The given sample shows 2 identical labels except that label "Test2" has LineHeight=2.

Under Android, LineHeight only affects multi-line spacing (it correctly has no effect if there is only a single line of text).
Under iOS, LineHeight changes layout even when there is only a single line.

Regardless of which one if correct, the behavior should be the same under Android and iOS. It seems to me that the Android behavior is the correct one.

Steps to Reproduce

  1. Run the repro sample on Android and IOS. Android produces the correct result:
    image
    iOS produces the wrong result:
    image

layoutbug12.zip

Version with bug

Release Candidate 3 (current)

Last version that worked well

Unknown/Other

Affected platforms

iOS

Affected platform versions

iOS 15

Did you find any workaround?

no

Relevant log output

No response

@philipag philipag added s/needs-verification Indicates that this issue needs initial verification before further triage will happen t/bug Something isn't working labels May 16, 2022
@jsuarezruiz jsuarezruiz added the area-controls-label Label, Span label May 16, 2022
@ghost ghost added the legacy-area-controls Label, Button, CheckBox, Slider, Stepper, Switch, Picker, Entry, Editor label May 16, 2022
@jsuarezruiz jsuarezruiz added platform/iOS 🍎 and removed legacy-area-controls Label, Button, CheckBox, Slider, Stepper, Switch, Picker, Entry, Editor labels May 16, 2022
@jfversluis jfversluis added area-layout StackLayout, GridLayout, ContentView, AbsoluteLayout, FlexLayout, ContentPresenter and removed s/needs-verification Indicates that this issue needs initial verification before further triage will happen labels May 16, 2022
@v-longmin v-longmin added s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage labels May 17, 2022
@v-longmin
Copy link

Verified repro on iOS 15.4, not repro on Android 12.0 with VS 17.3.0 Preview 1.0 [32427.505.main]. Repro project:
layoutbug12.zip

@Redth Redth added this to the 6.0.300-servicing milestone May 17, 2022
@Redth Redth modified the milestones: 6.0-servicing, Backlog Aug 30, 2022
@ghost
Copy link

ghost commented Aug 30, 2022

We've moved this issue to the Backlog milestone. This means that it is not going to be worked on for the coming release. We will reassess the backlog following the current release and consider this item at that time. To learn more about our issue management process and to have better expectation regarding different types of issues you can read our Triage Process.

@samhouts samhouts removed s/verified Verified / Reproducible Issue ready for Engineering Triage s/triaged Issue has been reviewed labels Apr 5, 2023
@XamlTest XamlTest added s/verified Verified / Reproducible Issue ready for Engineering Triage s/triaged Issue has been reviewed labels May 5, 2023
@XamlTest
Copy link
Collaborator

XamlTest commented May 5, 2023

Verified this on Visual Studio Enterprise 17.6.0 Preview 6.0. Repro on iOS 16.4, but does not repro on Android 13 with below Project: 7222.zip

iOS:
image

Android:
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-controls-label Label, Span area-layout StackLayout, GridLayout, ContentView, AbsoluteLayout, FlexLayout, ContentPresenter platform/iOS 🍎 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

7 participants