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

LineHeight property on label affects "invisible" line 0 and pushes all other texts down #11998

Closed
innomotionmedia opened this issue Dec 9, 2022 · 2 comments
Labels
area-controls-label Label, Span platform/android 🤖 platform/iOS 🍎 platform/macOS 🍏 macOS / Mac Catalyst platform/windows 🪟 s/needs-repro Attach a solution or code which reproduces the issue t/bug Something isn't working

Comments

@innomotionmedia
Copy link

innomotionmedia commented Dec 9, 2022

Description

grafik

Steps to Reproduce

Create a label and give it a line height:

Now, you see in the image, that the label does not sit in the middle of the background anymore. It would, if I were to remove the line height attribute. So even with liners of labels, they all get pushed down as if there would be a 0th line at the top.

This is clearly a bug in the loop that goes through each line and sepperates them

Link to public reproduction project repository

na

Version with bug

7.0 (current)

Last version that worked well

Unknown/Other

Affected platforms

iOS, Android, Windows, macOS, Other (Tizen, Linux, etc. not supported by Microsoft directly)

Affected platform versions

all

Did you find any workaround?

no

Relevant log output

No response

@innomotionmedia innomotionmedia added the t/bug Something isn't working label Dec 9, 2022
@Eilon Eilon added the legacy-area-controls Label, Button, CheckBox, Slider, Stepper, Switch, Picker, Entry, Editor label Dec 9, 2022
@jfversluis jfversluis added the s/needs-repro Attach a solution or code which reproduces the issue label Dec 12, 2022
@ghost
Copy link

ghost commented Dec 12, 2022

Hi @innomotionmedia. We have added the "s/needs-repro" label to this issue, which indicates that we require steps and sample code to reproduce the issue before we can take further action. Please try to create a minimal sample project/solution or code samples which reproduce the issue, ideally as a GitHub repo that we can clone. See more details about creating repros here: https://github.com/dotnet/maui/blob/main/.github/repro.md

This issue will be closed automatically in 7 days if we do not hear back from you by then - please feel free to re-open it if you come back to this issue after that time.

@ghost
Copy link

ghost commented Dec 16, 2022

This issue has been automatically marked as stale because it has been marked as requiring author feedback to reproduce the issue but has not had any activity for 4 days. It will be closed if no further activity occurs within 3 days of this comment. If it is closed, feel free to comment when you are able to provide the additional information and we will re-investigate.

@ghost ghost added the s/no-recent-activity Issue has had no recent activity label Dec 16, 2022
@ghost ghost closed this as completed Dec 19, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Jan 18, 2023
@ghost ghost removed the s/no-recent-activity Issue has had no recent activity label Feb 1, 2024
@Eilon Eilon added area-controls-label Label, Span and removed legacy-area-controls Label, Button, CheckBox, Slider, Stepper, Switch, Picker, Entry, Editor labels May 13, 2024
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-controls-label Label, Span platform/android 🤖 platform/iOS 🍎 platform/macOS 🍏 macOS / Mac Catalyst platform/windows 🪟 s/needs-repro Attach a solution or code which reproduces the issue t/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants