Components: Persist withFocusOutside event for async usage

isBlurring instance variable was never used
WordPress · Nov 30, 2017 · 1efbb9f · 1efbb9f
1 parent 467f044
commit 1efbb9f
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/components/higher-order/with-focus-outside/index.js b/components/higher-order/with-focus-outside/index.js
@@ -27,7 +27,10 @@ function withFocusOutside( WrappedComponent ) {
 		}
 
 		queueBlurCheck( event ) {
-			this.isBlurring = true;
+			// React does not allow using an event reference asynchronously
+			// due to recycling behavior, except when explicitly persisted.
+			event.persist();
+
 			this.blurCheckTimeout = setTimeout( () => {
 				if ( 'function' === typeof this.node.handleFocusOutside ) {
 					this.node.handleFocusOutside( event );
@@ -36,7 +39,6 @@ function withFocusOutside( WrappedComponent ) {
 		}
 
 		cancelBlurCheck() {
-			delete this.isBlurring;
 			clearTimeout( this.blurCheckTimeout );
 		}