use same height with or without avatar

Expensify · Sep 25, 2024 · 92e5979 · 92e5979
1 parent 7875e1d
commit 92e5979
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/components/ReportActionItem/TaskPreview.tsx b/src/components/ReportActionItem/TaskPreview.tsx
@@ -74,8 +74,9 @@ function TaskPreview({taskReportID, action, contextMenuAnchor, chatReportID, che
     const taskAssigneeAccountID = Task.getTaskAssigneeAccountID(taskReport) ?? action?.childManagerAccountID ?? -1;
     const personalDetails = usePersonalDetails();
     const avatar = personalDetails?.[taskAssigneeAccountID]?.avatar ?? Expensicons.FallbackAvatar;
+    const avatarSize = CONST.AVATAR_SIZE.SMALL;
     const isDeletedParentAction = ReportUtils.isCanceledTaskReport(taskReport, action);
-    const iconWrapperStyle = StyleUtils.getTaskPreviewIconWrapper(taskAssigneeAccountID > 0);
+    const iconWrapperStyle = StyleUtils.getTaskPreviewIconWrapper(avatarSize);
 
     if (isDeletedParentAction) {
         return <RenderHTML html={`<comment>${translate('parentReportAction.deletedTask')}</comment>`} />;
@@ -113,7 +114,7 @@ function TaskPreview({taskReportID, action, contextMenuAnchor, chatReportID, che
                         <Avatar
                             containerStyles={[styles.mr2, isTaskCompleted ? styles.opacitySemiTransparent : undefined]}
                             source={avatar}
-                            size={CONST.AVATAR_SIZE.SMALL}
+                            size={avatarSize}
                             avatarID={taskAssigneeAccountID}
                             type={CONST.ICON_TYPE_AVATAR}
                         />

diff --git a/src/styles/utils/index.ts b/src/styles/utils/index.ts
@@ -1669,10 +1669,9 @@ const createStyleUtils = (theme: ThemeColors, styles: ThemeStyles) => ({
         justifyContent: 'center',
     }),
 
-    getTaskPreviewIconWrapper: (hasAvatar: boolean) => {
-        const height = hasAvatar ? getAvatarSize(CONST.AVATAR_SIZE.SMALL) : variables.fontSizeNormalHeight;
+    getTaskPreviewIconWrapper: (avatarSize: AvatarSizeName) => {
         return {
-            height,
+            height: getAvatarSize(avatarSize),
             ...styles.justifyContentCenter,
         };
     },