Don't use window global

WordPress · Jun 21, 2024 · 285ac09 · 285ac09
1 parent 627a0cd
commit 285ac09
Showing 1 changed file with 13 additions and 3 deletions.
diff --git a/packages/block-editor/src/utils/dom.js b/packages/block-editor/src/utils/dom.js
@@ -95,7 +95,12 @@ function rectIntersect( rect1, rect2 ) {
  * @return {boolean} Whether the element is visible.
  */
 function isElementVisible( element ) {
-	const style = window.getComputedStyle( element );
+	const viewport = element.ownerDocument.defaultView;
+	if ( ! viewport ) {
+		return false;
+	}
+
+	const style = viewport.getComputedStyle( element );
 	if (
 		style.display === 'none' ||
 		style.visibility === 'hidden' ||
@@ -140,6 +145,11 @@ function isElementVisible( element ) {
  * @return {DOMRect} Bounding client rect.
  */
 export function getVisibleBoundingRect( element ) {
+	const viewport = element.ownerDocument.defaultView;
+	if ( ! viewport ) {
+		return new window.DOMRect();
+	}
+
 	let bounds = element.getBoundingClientRect();
 
 	const stack = [ element ];
@@ -158,8 +168,8 @@ export function getVisibleBoundingRect( element ) {
 	const viewportRect = new window.DOMRect(
 		0,
 		0,
-		window.innerWidth,
-		window.innerHeight
+		viewport.innerWidth,
+		viewport.innerHeight
 	);
 	return rectIntersect( bounds, viewportRect );
 }