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-25161] iOS: Add thin, light, ultra-light & semi-bold font-sizes #9308
Conversation
@hansemannn Can you resolve conflict here. Thanks. |
iphone/Classes/WebFont.m
Outdated
if (self.isSemiboldWeight) { | ||
font = [[UIFont systemFontOfSize:self.size weight:UIFontWeightSemibold] retain]; | ||
} else if (self.isItalicStyle) { | ||
font = [[UIFont italicSystemFontOfSize:self.size] retain]; |
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.
Why put this api inside block of iOS 8.2 and greater ?
iphone/Classes/WebFont.m
Outdated
didChange |= (self.isBoldWeight)||(self.isSemiboldWeight)||!(self.isNormalWeight); | ||
if ([fontWeightObject isEqualToString:@"semibold"]) { | ||
didChange |= !(self.isSemiboldWeight)||(self.isBoldWeight)||(self.isNormalWeight)||(self.isThinWeight)||(self.isLightWeight)||(self.isUltraLightWeight); | ||
self.isSemiboldWeight = YES; |
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.
fix indentation.
1748a65
to
cddd311
Compare
@vijaysingh-axway We linted the iOS source yesterday night, so in case you are wondering why the line numbers differ: That's why. For your PR's (which will also all have merge conflicts), you can just pull from master, use your local version for resolving the conflicts and call |
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.
CR passed.
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.
FR Passed.
New font weights are available and usable without issue.
JIRA: https://jira.appcelerator.org/browse/TIMOB-25161