Components: refactor Modal to pass exhaustive-deps (#41610)

chad1008 · web-flow · commit 00e413e0e9ae · 2022-06-15T05:41:03.000-04:00
* Modal: add `bodyOpenClassName` to `useEffect` deps

* Modal: update changelog
diff --git a/packages/components/CHANGELOG.md b/packages/components/CHANGELOG.md
@@ -27,6 +27,8 @@
 -   Enhance the TypeScript migration guidelines ([#41669](https://github.com/WordPress/gutenberg/pull/41669)).
 -   `ExternalLink`: Convert to TypeScript ([#41681](https://github.com/WordPress/gutenberg/pull/41681)).
 -   `InputControl` updated to satisfy `react/exhuastive-deps` eslint rule ([#41601](https://github.com/WordPress/gutenberg/pull/41601))
+-   `Modal`: updated to satisfy `react/exhuastive-deps` eslint rule ([#41610](https://github.com/WordPress/gutenberg/pull/41610))
+
 
 ### Experimental
 
diff --git a/packages/components/src/modal/index.js b/packages/components/src/modal/index.js
@@ -89,7 +89,7 @@ function Modal( props, forwardedRef ) {
 				ariaHelper.showApp();
 			}
 		};
-	}, [] );
+	}, [ bodyOpenClassName ] );
 
 	function handleEscapeKeyDown( event ) {
 		if (

Original file line number	Diff line number	Diff line change
`@@ -89,7 +89,7 @@ function Modal( props, forwardedRef ) {`
`89`	`89`	`ariaHelper.showApp();`
`90`	`90`	`}`
`91`	`91`	`};`
`92`		`- }, [] );`
	`92`	`+ }, [ bodyOpenClassName ] );`
`93`	`93`
`94`	`94`	`function handleEscapeKeyDown( event ) {`
`95`	`95`	`if (`