Merge pull request #14304 from Expensify/cmartins-alignBaseline

Fix display name and date alignment
Expensify · Jan 30, 2023 · 83371f7 · 83371f7
2 parents 5571e81 + ac94ca3
commit 83371f7
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 6 deletions.
diff --git a/src/pages/home/report/ReportActionItem.js b/src/pages/home/report/ReportActionItem.js
@@ -157,6 +157,10 @@ class ReportActionItem extends Component {
                 />
             );
         } else {
+            const message = _.last(lodashGet(this.props.action, 'message', [{}]));
+            const isAttachment = _.has(this.props.action, 'isAttachment')
+                ? this.props.action.isAttachment
+                : ReportUtils.isReportMessageAttachment(message);
             children = (
                 <ShowContextMenuContext.Provider
                     value={{
@@ -168,7 +172,10 @@ class ReportActionItem extends Component {
                 >
                     {!this.props.draftMessage
                         ? (
-                            <ReportActionItemMessage action={this.props.action} />
+                            <ReportActionItemMessage
+                                action={this.props.action}
+                                style={(!this.props.displayAsGroup && isAttachment) ? [styles.mt2] : undefined}
+                            />
                         ) : (
                             <ReportActionItemMessageEdit
                                 action={this.props.action}

diff --git a/src/styles/styles.js b/src/styles/styles.js
@@ -1320,25 +1320,21 @@ const styles = {
         fontSize: variables.fontSizeNormal,
         fontWeight: fontWeightBold,
         lineHeight: variables.lineHeightXLarge,
-        paddingBottom: 4,
         ...wordBreak.breakWord,
     },
 
     chatItemMessageHeaderTimestamp: {
         flexShrink: 0,
         color: themeColors.textSupporting,
         fontSize: variables.fontSizeSmall,
-        height: 24,
-        lineHeight: variables.lineHeightXLarge,
+        paddingTop: 2,
     },
 
     chatItemMessage: {
         color: themeColors.text,
         fontSize: variables.fontSizeNormal,
         fontFamily: fontFamily.EXP_NEUE,
         lineHeight: variables.lineHeightXLarge,
-        marginTop: -2,
-        marginBottom: -2,
         maxWidth: '100%',
         cursor: 'auto',
         ...whiteSpace.preWrap,