fixed bug where mini context menu wouldn't close

Expensify · Mar 6, 2023 · 5e7e7c1 · 5e7e7c1
1 parent db1026a
commit 5e7e7c1
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 2 deletions.
diff --git a/src/components/Reactions/MiniQuickEmojiReactions.js b/src/components/Reactions/MiniQuickEmojiReactions.js
@@ -25,8 +25,21 @@ const ICON_SIZE_SCALE_FACTOR = 1.3;
 
 const propTypes = {
     ...baseQuickEmojiReactionsPropTypes,
+
+    /**
+     * Will be called when the user closed the emoji picker
+     * without selecting an emoji.
+     */
+    onEmojiPickerClosed: PropTypes.func,
+
     ...withLocalizePropTypes,
     preferredSkinTone: PropTypes.oneOfType([PropTypes.number, PropTypes.string]).isRequired,
+
+};
+
+const defaultProps = {
+    onEmojiPickerClosed: () => {},
+    ...baseQuickEmojiReactionsDefaultProps,
 };
 
 /**
@@ -43,7 +56,7 @@ const MiniQuickEmojiReactions = (props) => {
     const openEmojiPicker = () => {
         props.onPressOpenPicker();
         EmojiPickerAction.showEmojiPicker(
-            () => {},
+            props.onEmojiPickerClosed,
             (emojiCode, emojiObject) => {
                 props.onEmojiSelected(emojiObject);
             },
@@ -89,7 +102,7 @@ const MiniQuickEmojiReactions = (props) => {
 
 MiniQuickEmojiReactions.displayName = 'MiniQuickEmojiReactions';
 MiniQuickEmojiReactions.propTypes = propTypes;
-MiniQuickEmojiReactions.defaultProps = baseQuickEmojiReactionsDefaultProps;
+MiniQuickEmojiReactions.defaultProps = defaultProps;
 export default compose(
     withLocalize,
     withOnyx({

diff --git a/src/pages/home/report/ContextMenu/ContextMenuActions.js b/src/pages/home/report/ContextMenu/ContextMenuActions.js
@@ -68,6 +68,7 @@ export default [
                         key="MiniQuickEmojiReactions"
                         onEmojiSelected={onEmojiSelected}
                         onPressOpenPicker={keepOpen}
+                        onEmojiPickerClosed={closeContextMenu}
                     />
                 );
             }