Node can be null

WordPress · May 21, 2021 · eaab6c1 · eaab6c1
1 parent b700c0e
commit eaab6c1
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/packages/compose/README.md b/packages/compose/README.md
@@ -399,7 +399,7 @@ _Parameters_
 
 _Returns_
 
--   `import('react').RefCallback<Node>`: Ref callback.
+-   `import('react').RefCallback<Node | null>`: Ref callback.
 
 <a name="useResizeObserver" href="#useResizeObserver">#</a> **useResizeObserver**
 

diff --git a/packages/compose/src/hooks/use-ref-effect/index.js b/packages/compose/src/hooks/use-ref-effect/index.js
@@ -20,12 +20,12 @@ import { useCallback, useRef } from '@wordpress/element';
  * @param {(node: Node) => (() => void) | undefined} callback     Callback with ref as argument.
  * @param {import('react').DependencyList}    dependencies Dependencies of the callback.
  *
- * @return {import('react').RefCallback<Node>} Ref callback.
+ * @return {import('react').RefCallback<Node | null>} Ref callback.
  */
 export default function useRefEffect( callback, dependencies ) {
 	/** @type {import('react').MutableRefObject<(() => void) | undefined>}  */
 	const cleanup = useRef();
-	return useCallback( ( /** @type {Node} */ node ) => {
+	return useCallback( ( /** @type {Node | null} */ node ) => {
 		if ( node ) {
 			cleanup.current = callback( node );
 		} else if ( cleanup.current ) {