diff --git a/app/components/emoji/emoji.js b/app/components/emoji/emoji.js index 388f1fb24..182439045 100644 --- a/app/components/emoji/emoji.js +++ b/app/components/emoji/emoji.js @@ -110,16 +110,6 @@ export default class Emoji extends React.PureComponent { const width = size; const height = size; - let marginTop = 0; - if (textStyle) { - // hack to get the vertical alignment looking better - if (fontSize > 16) { - marginTop -= 2; - } else if (fontSize <= 16) { - marginTop += 1; - } - } - // Android can't change the size of an image after its first render, so // force a new image to be rendered when the size changes const key = Platform.OS === 'android' ? (height + '-' + width) : null; @@ -128,7 +118,7 @@ export default class Emoji extends React.PureComponent { return ( ); } @@ -136,7 +126,7 @@ export default class Emoji extends React.PureComponent { return ( diff --git a/app/components/file_attachment_list/file_attachment_list.js b/app/components/file_attachment_list/file_attachment_list.js index d6503ae4d..b52385f86 100644 --- a/app/components/file_attachment_list/file_attachment_list.js +++ b/app/components/file_attachment_list/file_attachment_list.js @@ -8,7 +8,6 @@ import { Platform, ScrollView, StyleSheet, - View, } from 'react-native'; import {Client4} from 'mattermost-redux/client'; @@ -167,23 +166,18 @@ export default class FileAttachmentList extends Component { const {fileIds, isFailed} = this.props; return ( - - 1} - style={[styles.flex, (isFailed && styles.failed)]} - > - {this.renderItems()} - - + 1} + style={[(isFailed && styles.failed)]} + > + {this.renderItems()} + ); } } const styles = StyleSheet.create({ - flex: { - flex: 1, - }, failed: { opacity: 0.5, }, diff --git a/app/components/network_indicator/network_indicator.js b/app/components/network_indicator/network_indicator.js index b513d43a0..b7bf177b5 100644 --- a/app/components/network_indicator/network_indicator.js +++ b/app/components/network_indicator/network_indicator.js @@ -36,7 +36,6 @@ const { IOS_TOP_LANDSCAPE, IOS_TOP_PORTRAIT, IOSX_TOP_PORTRAIT, - STATUS_BAR_HEIGHT, } = ViewTypes; export default class NetworkIndicator extends PureComponent { @@ -70,6 +69,7 @@ export default class NetworkIndicator extends PureComponent { const navBar = this.getNavBarHeight(props.isLandscape); this.top = new Animated.Value(navBar - HEIGHT); + this.opacity = 0; this.backgroundColor = new Animated.Value(0); this.firstRun = true; @@ -169,6 +169,7 @@ export default class NetworkIndicator extends PureComponent { ), ]).start(() => { this.backgroundColor.setValue(0); + this.opacity = 0; }); }; @@ -188,7 +189,7 @@ export default class NetworkIndicator extends PureComponent { } else if (isX) { return IOSX_TOP_PORTRAIT; } else if (isLandscape) { - return IOS_TOP_LANDSCAPE + STATUS_BAR_HEIGHT; + return IOS_TOP_LANDSCAPE; } return IOS_TOP_PORTRAIT; @@ -299,6 +300,8 @@ export default class NetworkIndicator extends PureComponent { }; show = () => { + this.opacity = 1; + Animated.timing( this.top, { toValue: this.getNavBarHeight(), @@ -318,7 +321,6 @@ export default class NetworkIndicator extends PureComponent { let i18nId; let defaultMessage; - let values; let action; if (isOnline) { @@ -358,12 +360,11 @@ export default class NetworkIndicator extends PureComponent { } return ( - + {action} @@ -394,12 +395,6 @@ const styles = StyleSheet.create({ fontWeight: '600', flex: 1, }, - actionButton: { - alignItems: 'center', - borderWidth: 1, - borderColor: '#FFFFFF', - paddingRight: 0, - }, actionContainer: { alignItems: 'flex-end', height: 24, diff --git a/app/components/post/post.js b/app/components/post/post.js index 342e1dbbf..018783fdc 100644 --- a/app/components/post/post.js +++ b/app/components/post/post.js @@ -311,7 +311,7 @@ export default class Post extends PureComponent { return ( { return { + postStyle: { + overflow: 'hidden', + }, container: { flexDirection: 'row', - overflow: 'hidden', }, pendingPost: { opacity: 0.5, @@ -379,13 +381,6 @@ const getStyleSheet = makeStyleSheetFromTheme((theme) => { marginLeft: 47, marginTop: 10, }, - consecutivePostWithFlag: { - width: 11, - height: 11, - position: 'absolute', - top: -6, - left: -11, - }, profilePictureContainer: { marginBottom: 5, marginRight: 10, diff --git a/app/components/post_body/post_body.js b/app/components/post_body/post_body.js index 8216ea0c1..7f372d24c 100644 --- a/app/components/post_body/post_body.js +++ b/app/components/post_body/post_body.js @@ -190,17 +190,15 @@ export default class PostBody extends PureComponent { return ( - - - + ); }; @@ -341,17 +339,15 @@ export default class PostBody extends PureComponent { const {allUserIds, allUsernames, messageData} = postProps.user_activity; messageComponent = ( - - - + ); } else if (isEmojiOnly) { @@ -369,7 +365,7 @@ export default class PostBody extends PureComponent { messageComponent = ( - - - {body} - + + {body} {isFailed && { return { - flex: { - flex: 1, - overflow: 'hidden', - }, row: { flexDirection: 'row', }, @@ -459,7 +449,6 @@ const getStyleSheet = makeStyleSheetFromTheme((theme) => { }, messageContainerWithReplyBar: { flexDirection: 'row', - flex: 1, }, pendingPost: { opacity: 0.5, diff --git a/app/components/post_body_additional_content/post_body_additional_content.js b/app/components/post_body_additional_content/post_body_additional_content.js index 6f739b06b..2bd38e323 100644 --- a/app/components/post_body_additional_content/post_body_additional_content.js +++ b/app/components/post_body_additional_content/post_body_additional_content.js @@ -506,7 +506,6 @@ export default class PostBodyAdditionalContent extends PureComponent { const styles = StyleSheet.create({ imageContainer: { alignItems: 'flex-start', - flex: 1, justifyContent: 'flex-start', marginBottom: 6, marginTop: 10, diff --git a/app/components/reactions/reactions.js b/app/components/reactions/reactions.js index e8d2d9399..e29f83908 100644 --- a/app/components/reactions/reactions.js +++ b/app/components/reactions/reactions.js @@ -7,7 +7,6 @@ import { Image, ScrollView, TouchableOpacity, - View, } from 'react-native'; import {intlShape} from 'react-intl'; import MaterialIcon from 'react-native-vector-icons/MaterialIcons'; @@ -168,24 +167,19 @@ export default class Reactions extends PureComponent { } return ( - - - {reactionElements} - - + + {reactionElements} + ); } } const getStyleSheet = makeStyleSheetFromTheme((theme) => { return { - container: { - flex: 1, - }, addReaction: { tintColor: changeOpacity(theme.centerChannelColor, 0.5), width: 23,