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
[TIMOB-25971] : iOS minimumFontSize property in Ti.UI.label lowers text baseline as it decreases #10006
Conversation
…xt baseline as it decreases
Generated by 🚫 dangerJS |
…xt baseline as it decreases
@@ -463,6 +463,7 @@ - (void)setMinimumFontSize_:(id)size | |||
[label setAdjustsFontSizeToFitWidth:YES]; | |||
CGFloat ratio = minFontSize / label.font.pointSize; | |||
[label setMinimumScaleFactor:ratio]; | |||
[label setBaselineAdjustment:UIBaselineAdjustmentNone]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good fix! I am wondering what the baseline-adjustment is before setting this? Since UIBaselineAdjustmentNone
is already the default. And should we expose this as a property as well? Are there use-cases in our environment for this? And Android parity? Definitely as an own ticket, but maybe not even relevant.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Default is UIBaselineAdjustmentAlignBaselines. This property is only relevant with "adjustsFontSizeToFitWidth" = true. I don't think it is more relevant to expose, if we have already expose property minimumFontSize which includes minimumScaleFactor and adjustsFontSizeToFitWidth.
Generated by 🚫 dangerJS |
https://jira.appcelerator.org/browse/TIMOB-25971