From 9ab22e24c2cb57dee65b3bb438d685fa55c9234b Mon Sep 17 00:00:00 2001 From: Ovgodd Date: Wed, 7 Oct 2026 15:04:00 +0200 Subject: [PATCH] =?UTF-8?q?fixup!=20=E2=9C=A8(frontend)=20add=20open-in-se?= =?UTF-8?q?parate-window=20control=20for=20screen=20shares?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../rooms/livekit/components/ScreenShareFullscreenButton.tsx | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/frontend/src/features/rooms/livekit/components/ScreenShareFullscreenButton.tsx b/src/frontend/src/features/rooms/livekit/components/ScreenShareFullscreenButton.tsx index 21fb80656..2029e0242 100644 --- a/src/frontend/src/features/rooms/livekit/components/ScreenShareFullscreenButton.tsx +++ b/src/frontend/src/features/rooms/livekit/components/ScreenShareFullscreenButton.tsx @@ -27,6 +27,9 @@ export const ScreenShareFullscreenButton = memo( // Covers Esc and browser UI exits, not just this button. // Listens on the element's own document, so it still works in the popup. // Only this tile's instance announces to avoid duplicates with multiple shares. + // Opening the separate window passes another ref, the popup's. That is + // what re-runs this effect and moves the listener there. `.current` + // changing would not. useEffect(() => { const doc = getOwnerDocument(containerRef.current) setIsFullscreenAvailable(doc.fullscreenEnabled)