From c2d87ef88dfdc18b19cd0f3ed132c35a857753d3 Mon Sep 17 00:00:00 2001 From: Prashant Mangukiya Date: Sat, 18 Dec 2021 10:54:58 +0530 Subject: [PATCH 1/2] Default cursor set for IOU Preview --- src/components/ReportActionItem/IOUPreview.js | 10 +++++++++- src/pages/iou/IOUDetailsModal.js | 1 + src/styles/styles.js | 4 ++++ 3 files changed, 14 insertions(+), 1 deletion(-) diff --git a/src/components/ReportActionItem/IOUPreview.js b/src/components/ReportActionItem/IOUPreview.js index d7f21e9abbb4..190feee3177b 100644 --- a/src/components/ReportActionItem/IOUPreview.js +++ b/src/components/ReportActionItem/IOUPreview.js @@ -35,6 +35,12 @@ const propTypes = { /** The associated chatReport */ chatReportID: PropTypes.number.isRequired, + /** Callback for the preview pressed */ + onPreviewPressed: PropTypes.func, + + /** Extra styles to pass to View wrapper */ + containerStyles: PropTypes.arrayOf(PropTypes.object), + /* Onyx Props */ /** Active IOU Report for current report */ @@ -72,6 +78,8 @@ const defaultProps = { iouReport: {}, shouldHidePayButton: false, onPayButtonPressed: null, + onPreviewPressed: null, + containerStyles: [], }; const IOUPreview = (props) => { @@ -103,7 +111,7 @@ const IOUPreview = (props) => { const cachedTotal = props.iouReport.cachedTotal ? props.iouReport.cachedTotal.replace(/[()]/g, '') : ''; return ( - + {reportIsLoading ? : ( diff --git a/src/pages/iou/IOUDetailsModal.js b/src/pages/iou/IOUDetailsModal.js index 2cc2c0c407e0..e87034670eeb 100644 --- a/src/pages/iou/IOUDetailsModal.js +++ b/src/pages/iou/IOUDetailsModal.js @@ -115,6 +115,7 @@ class IOUDetailsModal extends Component { chatReportID={Number(this.props.route.params.chatReportID)} iouReportID={Number(this.props.route.params.iouReportID)} shouldHidePayButton + containerStyles={[styles.cursorDefault]} /> Date: Sat, 18 Dec 2021 17:31:04 +0530 Subject: [PATCH 2/2] Updates - Passing style.cursorPointer as need (instead of reset default cursor style) --- src/components/ReportActionItem/IOUAction.js | 2 ++ src/components/ReportActionItem/IOUPreview.js | 2 +- src/pages/iou/IOUDetailsModal.js | 1 - src/styles/styles.js | 5 ----- 4 files changed, 3 insertions(+), 7 deletions(-) diff --git a/src/components/ReportActionItem/IOUAction.js b/src/components/ReportActionItem/IOUAction.js index d172db8845fd..1baa2305b378 100644 --- a/src/components/ReportActionItem/IOUAction.js +++ b/src/components/ReportActionItem/IOUAction.js @@ -7,6 +7,7 @@ import reportActionPropTypes from '../../pages/home/report/reportActionPropTypes import IOUPreview from './IOUPreview'; import Navigation from '../../libs/Navigation/Navigation'; import ROUTES from '../../ROUTES'; +import styles from '../../styles/styles'; const propTypes = { /** All the data of the action */ @@ -50,6 +51,7 @@ const IOUAction = (props) => { chatReportID={props.chatReportID} onPayButtonPressed={launchDetailsModal} onPreviewPressed={launchDetailsModal} + containerStyles={[styles.cursorPointer]} /> )} diff --git a/src/components/ReportActionItem/IOUPreview.js b/src/components/ReportActionItem/IOUPreview.js index 190feee3177b..c1bbd411bf82 100644 --- a/src/components/ReportActionItem/IOUPreview.js +++ b/src/components/ReportActionItem/IOUPreview.js @@ -78,7 +78,7 @@ const defaultProps = { iouReport: {}, shouldHidePayButton: false, onPayButtonPressed: null, - onPreviewPressed: null, + onPreviewPressed: () => {}, containerStyles: [], }; diff --git a/src/pages/iou/IOUDetailsModal.js b/src/pages/iou/IOUDetailsModal.js index e87034670eeb..2cc2c0c407e0 100644 --- a/src/pages/iou/IOUDetailsModal.js +++ b/src/pages/iou/IOUDetailsModal.js @@ -115,7 +115,6 @@ class IOUDetailsModal extends Component { chatReportID={Number(this.props.route.params.chatReportID)} iouReportID={Number(this.props.route.params.iouReportID)} shouldHidePayButton - containerStyles={[styles.cursorDefault]} />