fix(@clayui/shared): fixes error when identifying non-visible element…

… as focusable
liferay · Oct 18, 2022 · 82cc0f8 · 82cc0f8
1 parent 1c2236e
commit 82cc0f8
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/packages/clay-shared/src/useFocusManagement.ts b/packages/clay-shared/src/useFocusManagement.ts
@@ -88,7 +88,13 @@ export const FOCUSABLE_ELEMENTS = [
 function collectDocumentFocusableElements() {
  return Array.from<HTMLElement>(
  document.querySelectorAll(FOCUSABLE_ELEMENTS.join(','))
- ).filter((element) => isFocusable(element));
+ ).filter((element) => {
+ if (isFocusable(element)) {
+ return window.getComputedStyle(element).visibility !== 'hidden';
+ }
+
+ return false;
+ });
 }
 
 // https://github.com/facebook/react/pull/15849#diff-39a673d38713257d5fe7d90aac2acb5aR107