fix: detect iframe DOM nodes

Co-authored-by: Codex <codex@openai.com>

Author: PinkChampagne17

src/Dom/findDOMNode.ts

@@ -1,9 +1,10 @@
-import React from 'react';
+import type React from 'react';
 
 export function isDOM(node: any): node is HTMLElement | SVGElement {
-  // https://developer.mozilla.org/en-US/docs/Web/API/Element
-  // Since XULElement is also subclass of Element, we only need HTMLElement and SVGElement
-  return node instanceof HTMLElement || node instanceof SVGElement;
+  // Cross-frame safe check: `instanceof` fails for elements created in a
+  // different frame (e.g. iframe via createPortal) because each frame has its
+  // own HTMLElement/SVGElement constructors.
+  return typeof node === 'object' && node !== null && node.nodeType === 1;
 }
 
 /**

src/Dom/isVisible.ts

@@ -3,7 +3,7 @@ export default (element: Element): boolean => {
     return false;
   }
 
-  if (element instanceof Element) {
+  if (element.nodeType === 1) {
     if ((element as HTMLElement).offsetParent) {
       return true;
     }

tests/findDOMNode.test.tsx

@@ -63,6 +63,27 @@ describe('findDOMNode', () => {
     expect(findDOMNode(svg)).toBe(svg);
   });
 
+  it('supports DOM nodes from iframe', () => {
+    const iframe = document.createElement('iframe');
+    document.body.appendChild(iframe);
+
+    try {
+      const iframeWindow = iframe.contentWindow!;
+      const iframeDocument = iframeWindow.document;
+      const div = iframeDocument.createElement('div');
+      iframeDocument.body.appendChild(div);
+
+      expect(div).toBeInstanceOf(iframeWindow.HTMLElement);
+      expect(div).not.toBeInstanceOf(HTMLElement);
+      expect(isDOM(div)).toBe(true);
+      expect(getDOM(div)).toBe(div);
+      expect(findDOMNode(div)).toBe(div);
+      expect(getDOM({ nativeElement: div })).toBe(div);
+    } finally {
+      document.body.removeChild(iframe);
+    }
+  });
+
   it('isDOM type', () => {
     const svg: any = document.createElementNS(
       'http://www.w3.org/2000/svg',

tests/isVisible.test.ts

@@ -0,0 +1,33 @@
+import isVisible from '../src/Dom/isVisible';
+
+describe('isVisible', () => {
+  it('supports DOM nodes from iframe', () => {
+    const iframe = document.createElement('iframe');
+    document.body.appendChild(iframe);
+
+    try {
+      const iframeWindow = iframe.contentWindow!;
+      const iframeDocument = iframeWindow.document;
+      const div = iframeDocument.createElement('div');
+      iframeDocument.body.appendChild(div);
+
+      jest.spyOn(div, 'getBoundingClientRect').mockReturnValue({
+        bottom: 10,
+        height: 10,
+        left: 0,
+        right: 10,
+        toJSON: () => {},
+        top: 0,
+        width: 10,
+        x: 0,
+        y: 0,
+      });
+
+      expect(div).toBeInstanceOf(iframeWindow.HTMLElement);
+      expect(div).not.toBeInstanceOf(Element);
+      expect(isVisible(div)).toBe(true);
+    } finally {
+      document.body.removeChild(iframe);
+    }
+  });
+});