backout (and remove) props.lineBreakMode from Text

Reviewed By: jingc

Differential Revision: D3063164

fb-gh-sync-id: 81776866c09bb06411b9660f1f4833ba515e046a
shipit-source-id: 81776866c09bb06411b9660f1f4833ba515e046a
This commit is contained in:
Michelle Hwang 2016-03-17 11:47:42 -07:00 committed by Facebook Github Bot 9
parent 3dbf9b9844
commit 14555063bb
6 changed files with 6 additions and 32 deletions

View File

@ -23,7 +23,6 @@ extern NSString *const RCTReactTagAttributeName;
@property (nonatomic, assign) BOOL isHighlighted;
@property (nonatomic, assign) CGFloat letterSpacing;
@property (nonatomic, assign) CGFloat lineHeight;
@property (nonatomic, assign) NSLineBreakMode lineBreakMode;
@property (nonatomic, assign) NSUInteger numberOfLines;
@property (nonatomic, assign) CGSize shadowOffset;
@property (nonatomic, assign) NSTextAlignment textAlign;

View File

@ -119,7 +119,7 @@ static css_dim_t RCTMeasure(void *context, float width, float height)
NSTextContainer *textContainer = [NSTextContainer new];
textContainer.lineFragmentPadding = 0.0;
textContainer.lineBreakMode = _lineBreakMode;
textContainer.lineBreakMode = _numberOfLines > 0 ? NSLineBreakByTruncatingTail : NSLineBreakByClipping;
textContainer.maximumNumberOfLines = _numberOfLines;
textContainer.size = (CGSize){isnan(width) ? CGFLOAT_MAX : width, CGFLOAT_MAX};
@ -370,7 +370,6 @@ RCT_TEXT_PROPERTY(FontStyle, _fontStyle, NSString *)
RCT_TEXT_PROPERTY(IsHighlighted, _isHighlighted, BOOL)
RCT_TEXT_PROPERTY(LetterSpacing, _letterSpacing, CGFloat)
RCT_TEXT_PROPERTY(LineHeight, _lineHeight, CGFloat)
RCT_TEXT_PROPERTY(LineBreakMode, _lineBreakMode, NSLineBreakMode)
RCT_TEXT_PROPERTY(NumberOfLines, _numberOfLines, NSUInteger)
RCT_TEXT_PROPERTY(TextAlign, _textAlign, NSTextAlignment)
RCT_TEXT_PROPERTY(TextDecorationColor, _textDecorationColor, UIColor *);

View File

@ -50,7 +50,6 @@ RCT_EXPORT_SHADOW_PROPERTY(fontStyle, NSString)
RCT_EXPORT_SHADOW_PROPERTY(isHighlighted, BOOL)
RCT_EXPORT_SHADOW_PROPERTY(letterSpacing, CGFloat)
RCT_EXPORT_SHADOW_PROPERTY(lineHeight, CGFloat)
RCT_EXPORT_SHADOW_PROPERTY(lineBreakMode, NSLineBreakMode)
RCT_EXPORT_SHADOW_PROPERTY(numberOfLines, NSUInteger)
RCT_EXPORT_SHADOW_PROPERTY(textAlign, NSTextAlignment)
RCT_EXPORT_SHADOW_PROPERTY(textDecorationStyle, NSUnderlineStyle)

View File

@ -219,18 +219,6 @@ const Text = React.createClass({
isHighlighted: this.state.isHighlighted,
};
}
if (Platform.OS === 'ios' && newProps.lineBreakMode === undefined) {
// Prevent mutation of `this.props`!
if (newProps === this.props) {
newProps = { ...this.props };
}
// If `numberOfLines` is undefined, it defaults to 0 in native code.
if (newProps.numberOfLines !== undefined && newProps.numberOfLines > 0) {
newProps.lineBreakMode = 'truncating-tail';
} else {
newProps.lineBreakMode = 'clipping';
}
}
if (this.context.isInAParentText) {
return <RCTVirtualText {...newProps} />;
} else {

View File

@ -40,17 +40,6 @@ var TextStylePropTypes = Object.assign(Object.create(ViewStylePropTypes), {
*/
letterSpacing: ReactPropTypes.number,
lineHeight: ReactPropTypes.number,
/**
* @platform ios
*/
lineBreakMode: ReactPropTypes.oneOf([
'clipping',
'word-wrapping',
'char-wrapping',
'truncating-head',
'truncating-middle',
'truncating-tail',
]),
/**
* Specifies text alignment. The value 'justify' is only supported on iOS.
*/

View File

@ -231,12 +231,12 @@ NSNumber *RCTConvertMultiEnumValue(const char *typeName, NSDictionary *mapping,
}
RCT_ENUM_CONVERTER(NSLineBreakMode, (@{
@"wordWrapping": @(NSLineBreakByWordWrapping),
@"charWrapping": @(NSLineBreakByCharWrapping),
@"clipping": @(NSLineBreakByClipping),
@"word-wrapping": @(NSLineBreakByWordWrapping),
@"char-wrapping": @(NSLineBreakByCharWrapping),
@"truncating-head": @(NSLineBreakByTruncatingHead),
@"truncating-middle": @(NSLineBreakByTruncatingMiddle),
@"truncating-tail": @(NSLineBreakByTruncatingTail),
@"truncatingHead": @(NSLineBreakByTruncatingHead),
@"truncatingTail": @(NSLineBreakByTruncatingTail),
@"truncatingMiddle": @(NSLineBreakByTruncatingMiddle),
}), NSLineBreakByWordWrapping, integerValue)
RCT_ENUM_CONVERTER(NSTextAlignment, (@{