Merge pull request #2364 from Expensify/Rory-FixSuccessIcon

Fix success icon in ReportActionContextMenu
Expensify · Apr 13, 2021 · f8ff884 · f8ff884
2 parents e572b16 + 44ffb0f
commit f8ff884
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/src/components/MenuItem.js b/src/components/MenuItem.js
@@ -24,11 +24,15 @@ const propTypes = {
 
     // Boolean whether to display the ArrowRight icon
     shouldShowRightArrow: PropTypes.bool,
+
+    // A boolean flag that gives the icon a green fill if true
+    success: PropTypes.bool,
 };
 
 const defaultProps = {
     shouldShowRightArrow: false,
     wrapperStyle: {},
+    success: false,
 };
 
 const MenuItem = ({
@@ -37,6 +41,7 @@ const MenuItem = ({
     title,
     shouldShowRightArrow,
     wrapperStyle,
+    success,
 }) => (
     <Pressable
         onPress={onPress}
@@ -50,7 +55,7 @@ const MenuItem = ({
             <>
                 <View style={styles.flexRow}>
                     <View style={styles.createMenuIcon}>
-                        <Icon src={icon} fill={getIconFillColor(getButtonState(hovered, pressed))} />
+                        <Icon src={icon} fill={getIconFillColor(getButtonState(hovered, pressed, success))} />
                     </View>
                     <View style={styles.justifyContentCenter}>
                         <Text style={[styles.createMenuText, styles.ml3]}>

diff --git a/src/pages/home/report/ReportActionContextMenuItem.js b/src/pages/home/report/ReportActionContextMenuItem.js
@@ -79,6 +79,7 @@ class ReportActionContextMenuItem extends Component {
                         icon={icon}
                         onPress={this.triggerPressAndUpdateSuccess}
                         wrapperStyle={styles.pr9}
+                        success={this.state.success}
                     />
                 )
         );