diff --git a/src/components/LHNOptionsList/OptionRowLHN.js b/src/components/LHNOptionsList/OptionRowLHN.js index 732051e6553e..2fae4b095e83 100644 --- a/src/components/LHNOptionsList/OptionRowLHN.js +++ b/src/components/LHNOptionsList/OptionRowLHN.js @@ -146,8 +146,8 @@ const OptionRowLHN = (props) => { backgroundColor={props.isFocused ? themeColors.activeComponentBG : themeColors.sidebar} mainAvatar={optionItem.icons[0]} secondaryAvatar={optionItem.icons[1]} - mainTooltip={optionItem.ownerEmail} - secondaryTooltip={optionItem.subtitle} + mainTooltip={optionItem.icons[0].name} + secondaryTooltip={optionItem.icons[1].name} size={props.viewMode === CONST.OPTION_MODE.COMPACT ? CONST.AVATAR_SIZE.SMALL : CONST.AVATAR_SIZE.DEFAULT} /> ) : ( @@ -160,7 +160,7 @@ const OptionRowLHN = (props) => { props.isFocused ? StyleUtils.getBackgroundAndBorderStyle(focusedBackgroundColor) : undefined, hovered && !props.isFocused ? StyleUtils.getBackgroundAndBorderStyle(hoveredBackgroundColor) : undefined, ]} - shouldShowTooltip={!optionItem.isChatRoom && !optionItem.isArchivedRoom} + shouldShowTooltip={!optionItem.isArchivedRoom} /> ))} diff --git a/src/components/MultipleAvatars.js b/src/components/MultipleAvatars.js index fa6330d4703f..8046c03c7699 100644 --- a/src/components/MultipleAvatars.js +++ b/src/components/MultipleAvatars.js @@ -64,7 +64,7 @@ const defaultProps = { const MultipleAvatars = (props) => { let avatarContainerStyles = props.size === CONST.AVATAR_SIZE.SMALL ? [styles.emptyAvatarSmall, styles.emptyAvatarMarginSmall] : [styles.emptyAvatar, styles.emptyAvatarMargin]; - const singleAvatarStyles = props.size === CONST.AVATAR_SIZE.SMALL ? styles.singleAvatarSmall : styles.singleAvatar; + const singleAvatarStyle = props.size === CONST.AVATAR_SIZE.SMALL ? styles.singleAvatarSmall : styles.singleAvatar; const secondAvatarStyles = [props.size === CONST.AVATAR_SIZE.SMALL ? styles.secondAvatarSmall : styles.secondAvatar, ...props.secondAvatarStyle]; const tooltipTexts = props.shouldShowTooltip ? _.pluck(props.icons, 'name') : []; @@ -172,7 +172,7 @@ const MultipleAvatars = (props) => { )} ) : ( - + {/* View is necessary for tooltip to show for multiple avatars in LHN */} @@ -180,13 +180,13 @@ const MultipleAvatars = (props) => { source={props.icons[0].source || props.fallbackIcon} fill={themeColors.iconSuccessFill} size={props.isFocusMode ? CONST.AVATAR_SIZE.MID_SUBSCRIPT : CONST.AVATAR_SIZE.SMALLER} - imageStyles={[singleAvatarStyles]} + imageStyles={[singleAvatarStyle]} name={props.icons[0].name} type={props.icons[0].type} /> - + {props.icons.length === 2 ? ( @@ -194,7 +194,7 @@ const MultipleAvatars = (props) => { source={props.icons[1].source || props.fallbackIcon} fill={themeColors.iconSuccessFill} size={props.isFocusMode ? CONST.AVATAR_SIZE.MID_SUBSCRIPT : CONST.AVATAR_SIZE.SMALLER} - imageStyles={[singleAvatarStyles]} + imageStyles={[singleAvatarStyle]} name={props.icons[1].name} type={props.icons[1].type} /> @@ -202,7 +202,7 @@ const MultipleAvatars = (props) => { ) : ( - +