Merge pull request #17286 from hasebsiddiqui/hasebsiddiqui-17182

Tooltip functionality behaving weird for the copy to clipboard
Expensify · Apr 14, 2023 · 1138093 · 1138093
2 parents ab9b7d9 + b8d3536
commit 1138093
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/src/components/ContextMenuItem.js b/src/components/ContextMenuItem.js
@@ -55,10 +55,9 @@ class ContextMenuItem extends Component {
      * Method to call parent onPress and toggleDelayButtonState
      */
     triggerPressAndUpdateSuccess() {
-        if (this.props.isDelayButtonStateComplete) {
-            return;
+        if (!this.props.isDelayButtonStateComplete) {
+            this.props.onPress();
         }
-        this.props.onPress();
 
         // We only set the success state when we have icon or text to represent the success state
         // We may want to replace this check by checking the Result from OnPress Callback in future.

diff --git a/src/components/withDelayToggleButtonState.js b/src/components/withDelayToggleButtonState.js
@@ -41,6 +41,11 @@ export default function (WrappedComponent) {
                 return;
             }
 
+            // Clear existing timer
+            if (this.resetButtonStateCompleteTimer) {
+                clearTimeout(this.resetButtonStateCompleteTimer);
+            }
+
             this.resetButtonStateCompleteTimer = setTimeout(() => {
                 this.setState({
                     isDelayButtonStateComplete: false,