From 6e7c669d226353337a97df7a1fda05157041454d Mon Sep 17 00:00:00 2001 From: Marco Ciampini Date: Wed, 28 Jun 2023 20:37:08 +0200 Subject: [PATCH] Remove comments --- packages/compose/src/hooks/use-focus-outside/index.ts | 5 ----- 1 file changed, 5 deletions(-) diff --git a/packages/compose/src/hooks/use-focus-outside/index.ts b/packages/compose/src/hooks/use-focus-outside/index.ts index 82870fe64406a..5a852934ff9e0 100644 --- a/packages/compose/src/hooks/use-focus-outside/index.ts +++ b/packages/compose/src/hooks/use-focus-outside/index.ts @@ -90,11 +90,6 @@ export default function useFocusOutside( } | null >( null ); const pollingIntervalId = useRef< number | undefined >(); - // Thoughts: - // - it needs to always stop when component unmounted - // - it needs to work when resuming focus from another doc and clicking - // immediately on the backdrop - // Sometimes the blur event is not reliable, for example when focus moves // to an iframe inside the wrapper. In these scenarios, we resort to polling, // and we explicitly check if focus has indeed moved outside the wrapper.