Skip to content

Commit 279836d

Browse files
authored
Merge pull request Expensify#60231 from bernhardoj/fix/59534-thump-up-icon-truncated
Fix approved icon is cut off
2 parents 7f1cdc3 + 7f68569 commit 279836d

2 files changed

Lines changed: 7 additions & 7 deletions

File tree

src/components/ReportActionItem/MoneyRequestReportPreview/MoneyRequestReportPreviewContent.tsx

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,7 @@ const checkIfReportNameOverflows = <T extends LayoutChangeEvent>({nativeEvent}:
8585
'target' in nativeEvent ? (nativeEvent as WebLayoutNativeEvent).target.scrollHeight > variables.h70 : false;
8686

8787
// Do not remove this empty view, it is a workaround for the icon padding at the end of the preview text
88-
const FixIconPadding = <View />;
88+
const FixIconPadding = <View style={{height: variables.iconSizeNormal}} />;
8989

9090
function MoneyRequestReportPreviewContent({
9191
iouReportID,
@@ -427,8 +427,8 @@ function MoneyRequestReportPreviewContent({
427427
<ImageSVG
428428
src={isApproved ? Expensicons.ThumbsUp : Expensicons.Checkmark}
429429
fill={isApproved ? theme.icon : theme.iconSuccessFill}
430-
width={20}
431-
height={20}
430+
width={variables.iconSizeNormal}
431+
height={variables.iconSizeNormal}
432432
style={{transform: 'translateY(4px)'}}
433433
contentFit="cover"
434434
/>
@@ -486,8 +486,8 @@ function MoneyRequestReportPreviewContent({
486486
<View style={[reportPreviewStyles.wrapperStyle]}>
487487
<View style={[reportPreviewStyles.contentContainerStyle]}>
488488
<View style={[styles.expenseAndReportPreviewTextContainer, styles.overflowHidden]}>
489-
<View style={[styles.flexRow, styles.justifyContentBetween, styles.gap3]}>
490-
<View style={[styles.flexRow, styles.mw100, styles.flexShrink1, (isApproved || iouSettled) && styles.mtn1]}>
489+
<View style={[styles.flexRow, styles.justifyContentBetween, styles.gap3, StyleUtils.getMinimumHeight(variables.h28)]}>
490+
<View style={[styles.flexRow, styles.mw100, styles.flexShrink1]}>
491491
<Animated.View style={[styles.flexRow, styles.alignItemsCenter, previewMessageStyle, styles.flexShrink1]}>
492492
<Text
493493
onLayout={onTextLayoutChange}

src/styles/index.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5649,8 +5649,8 @@ const styles = (theme: ThemeColors) =>
56495649
},
56505650
reportPreviewArrowButton: {
56515651
borderRadius: 50,
5652-
width: 28,
5653-
height: 28,
5652+
width: variables.w28,
5653+
height: variables.h28,
56545654
alignItems: 'center',
56555655
justifyContent: 'center',
56565656
marginLeft: 4,

0 commit comments

Comments
 (0)