Merge pull request Expensify#14507 from s77rt/hoverable-onblur

Fixed regression on Hoverable blur event
JediWattson · Jan 24, 2023 · 811dd58 · 811dd58
2 parents 6f70a7d + f133eb4
commit 811dd58
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/src/components/Hoverable/index.js b/src/components/Hoverable/index.js
@@ -85,7 +85,9 @@ class Hoverable extends Component {
                     }
                 },
                 onBlur: (el) => {
-                    this.setIsHovered(false);
+                    if (!this.wrapperView.contains(el.relatedTarget)) {
+                        this.setIsHovered(false);
+                    }
 
                     // Call the original onBlur, if any
                     const {onBlur} = this.props.children;
@@ -101,7 +103,12 @@ class Hoverable extends Component {
                 ref={el => this.wrapperView = el}
                 onMouseEnter={() => this.setIsHovered(true)}
                 onMouseLeave={() => this.setIsHovered(false)}
-                onBlur={() => this.setIsHovered(false)}
+                onBlur={(el) => {
+                    if (this.wrapperView.contains(el.relatedTarget)) {
+                        return;
+                    }
+                    this.setIsHovered(false);
+                }}
             >
                 { // If this.props.children is a function, call it to provide the hover state to the children.
                     _.isFunction(this.props.children)