fixed error with sorting values, fixing styles

Expensify · Dec 4, 2022 · 70c9ac3 · 70c9ac3
1 parent 480e550
commit 70c9ac3
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 6 deletions.
diff --git a/src/components/AttachmentCarousel/index.js b/src/components/AttachmentCarousel/index.js
@@ -96,10 +96,10 @@ class AttachmentCarousel extends React.Component {
      */
     makeStateWithReports() {
         let page;
-        const actionsArr = ReportActionsUtils.getSortedReportActions(this.props.reportActions);
-        const attachments = _.reduce(actionsArr, (attachmentsAccumulator, {action: reportAction}) => {
-            if (reportAction.originalMessage && reportAction.originalMessage.html) {
-                const matchesIt = reportAction.originalMessage.html.matchAll(CONST.REGEX.ATTACHMENT_DATA);
+        const actionsArr = ReportActionsUtils.getSortedReportActions(_.values(this.props.reportActions), true);
+        const attachments = _.reduce(actionsArr, (attachmentsAccumulator, {originalMessage}) => {
+            if (originalMessage && originalMessage.html) {
+                const matchesIt = originalMessage.html.matchAll(CONST.REGEX.ATTACHMENT_DATA);
                 const matches = [...matchesIt];
                 if (matches.length === 2) {
                     const [src, name] = matches;
@@ -169,7 +169,7 @@ class AttachmentCarousel extends React.Component {
                             style={[styles.leftAttachmentArrow]}
                             icon={Expensicons.BackArrow}
                             iconFill={themeColors.text}
-                            iconStyles={[styles.mr0, {PointerEvent: 'auto'}]}
+                            iconStyles={[styles.mr0]}
                             onPress={() => this.cycleThroughAttachments(1)}
                             isDisabled={this.state.isBackDisabled}
                         />
@@ -178,7 +178,7 @@ class AttachmentCarousel extends React.Component {
                             style={[styles.rightAttachmentArrow]}
                             icon={Expensicons.ArrowRight}
                             iconFill={themeColors.text}
-                            iconStyles={[styles.mr0, {PointerEvent: 'auto'}]}
+                            iconStyles={[styles.mr0]}
                             onPress={() => this.cycleThroughAttachments(-1)}
                             isDisabled={this.state.isForwardDisabled}
                         />

diff --git a/src/styles/styles.js b/src/styles/styles.js
@@ -2108,10 +2108,15 @@ const styles = {
     attachmentModalArrowsContainer: {
         display: 'flex',
         justifyContent: 'center',
+        cursor: 'unset',
         height: '100%',
         width: '100%',
     },
 
+    attachmentContainer: {
+        height: '100%',
+    },
+
     leftAttachmentArrow: {
         zIndex: 23,
         position: 'absolute',