FocusTrap: traverse arr instead of NodeList

VKCOM · Nov 19, 2021 · ab3395f · ab3395f
1 parent a08219a
commit ab3395f
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/components/FocusTrap/FocusTrap.tsx b/src/components/FocusTrap/FocusTrap.tsx
@@ -55,7 +55,8 @@ export const FocusTrap: React.FC<FocusTrapProps> = ({
 
     const nodes: HTMLElement[] = [];
     // eslint-disable-next-line no-restricted-properties
-    ref.current?.querySelectorAll(FOCUSABLE_ELEMENTS).forEach((focusableEl) => {
+    const allFocusableNodes = Array.from(ref.current.querySelectorAll(FOCUSABLE_ELEMENTS));
+    allFocusableNodes.forEach((focusableEl) => {
       const { display, visibility } = window.getComputedStyle(focusableEl);
 
       if (display !== 'none' && visibility !== 'hidden') {