Skip to content

Commit 3bc4d42

Browse files
authored
Merge pull request #1454 from amgleitman/dynamic-type-renames
A couple of renames to make Dynamic Type features more proper
2 parents d869ab1 + f0c6e13 commit 3bc4d42

File tree

4 files changed

+8
-8
lines changed

4 files changed

+8
-8
lines changed

Libraries/Text/RCTTextAttributes.m

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -234,7 +234,7 @@ - (CGFloat)effectiveFontSizeMultiplier
234234
#if !TARGET_OS_OSX // [TODO(macOS GH#774)
235235
if (_dynamicTypeRamp != RCTDynamicTypeRampUndefined) {
236236
UIFontMetrics *fontMetrics = RCTUIFontMetricsForDynamicTypeRamp(_dynamicTypeRamp);
237-
CGFloat baseSize = RCTUIBaseSizeForDynamicTypeRamp(_dynamicTypeRamp);
237+
CGFloat baseSize = RCTBaseSizeForDynamicTypeRamp(_dynamicTypeRamp);
238238
fontSizeMultiplier = [fontMetrics scaledValueForValue:baseSize] / baseSize;
239239
}
240240
#endif // ]TODO(macOS GH#774)

Libraries/Text/Text/RCTDynamicTypeRamp.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ typedef NS_ENUM(NSInteger, RCTDynamicTypeRamp) {
1616
RCTDynamicTypeRampCaption2,
1717
RCTDynamicTypeRampCaption1,
1818
RCTDynamicTypeRampFootnote,
19-
RCTDynamicTypeRampSubhead,
19+
RCTDynamicTypeRampSubheadline,
2020
RCTDynamicTypeRampCallout,
2121
RCTDynamicTypeRampBody,
2222
RCTDynamicTypeRampHeadline,
@@ -36,5 +36,5 @@ typedef NS_ENUM(NSInteger, RCTDynamicTypeRamp) {
3636
/// Generates a `UIFontMetrics` instance representing a particular Dynamic Type ramp.
3737
UIFontMetrics * _Nonnull RCTUIFontMetricsForDynamicTypeRamp(RCTDynamicTypeRamp dynamicTypeRamp);
3838
/// The "reference" size for a particular font scale ramp, equal to a text element's size under default text size settings.
39-
CGFloat RCTUIBaseSizeForDynamicTypeRamp(RCTDynamicTypeRamp dynamicTypeRamp);
39+
CGFloat RCTBaseSizeForDynamicTypeRamp(RCTDynamicTypeRamp dynamicTypeRamp);
4040
#endif // ]TODO(macOS GH#774)

Libraries/Text/Text/RCTDynamicTypeRamp.m

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ @implementation RCTConvert (DynamicTypeRamp)
1515
@"caption2": @(RCTDynamicTypeRampCaption2),
1616
@"caption1": @(RCTDynamicTypeRampCaption1),
1717
@"footnote": @(RCTDynamicTypeRampFootnote),
18-
@"subhead": @(RCTDynamicTypeRampSubhead),
18+
@"subheadline": @(RCTDynamicTypeRampSubheadline),
1919
@"callout": @(RCTDynamicTypeRampCallout),
2020
@"body": @(RCTDynamicTypeRampBody),
2121
@"headline": @(RCTDynamicTypeRampHeadline),
@@ -36,7 +36,7 @@ @implementation RCTConvert (DynamicTypeRamp)
3636
@(RCTDynamicTypeRampCaption2): UIFontTextStyleCaption2,
3737
@(RCTDynamicTypeRampCaption1): UIFontTextStyleCaption1,
3838
@(RCTDynamicTypeRampFootnote): UIFontTextStyleFootnote,
39-
@(RCTDynamicTypeRampSubhead): UIFontTextStyleSubheadline,
39+
@(RCTDynamicTypeRampSubheadline): UIFontTextStyleSubheadline,
4040
@(RCTDynamicTypeRampCallout): UIFontTextStyleCallout,
4141
@(RCTDynamicTypeRampBody): UIFontTextStyleBody,
4242
@(RCTDynamicTypeRampHeadline): UIFontTextStyleHeadline,
@@ -51,7 +51,7 @@ @implementation RCTConvert (DynamicTypeRamp)
5151
return [UIFontMetrics metricsForTextStyle:textStyle];
5252
}
5353

54-
CGFloat RCTUIBaseSizeForDynamicTypeRamp(RCTDynamicTypeRamp dynamicTypeRamp) {
54+
CGFloat RCTBaseSizeForDynamicTypeRamp(RCTDynamicTypeRamp dynamicTypeRamp) {
5555
static NSDictionary<NSNumber *, NSNumber *> *mapping;
5656
static dispatch_once_t onceToken;
5757
dispatch_once(&onceToken, ^{
@@ -60,7 +60,7 @@ CGFloat RCTUIBaseSizeForDynamicTypeRamp(RCTDynamicTypeRamp dynamicTypeRamp) {
6060
@(RCTDynamicTypeRampCaption2): @11,
6161
@(RCTDynamicTypeRampCaption1): @12,
6262
@(RCTDynamicTypeRampFootnote): @13,
63-
@(RCTDynamicTypeRampSubhead): @15,
63+
@(RCTDynamicTypeRampSubheadline): @15,
6464
@(RCTDynamicTypeRampCallout): @16,
6565
@(RCTDynamicTypeRampBody): @17,
6666
@(RCTDynamicTypeRampHeadline): @17,

Libraries/Text/TextProps.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -193,7 +193,7 @@ export type TextProps = $ReadOnly<{|
193193
| 'caption2'
194194
| 'caption1'
195195
| 'footnote'
196-
| 'subhead'
196+
| 'subheadline'
197197
| 'callout'
198198
| 'body'
199199
| 'headline'

0 commit comments

Comments
 (0)