github/oneuptime
1
1
Fork 0
mirror of https://github.com/OneUptime/oneuptime.git synced 2026-04-06 00:32:12 +02:00
Code Issues 193 Packages Projects Releases Wiki Activity

feat(modal): exclude close button from focus trap in modal

Browse Source
This commit is contained in:
Nawaz Dhandala
2026-01-26 19:43:07 +00:00
parent dd996539bc
commit 0bdf74cab2
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
Common/UI/Components/Modal/Modal.tsx
View File

@@ -66,9 +66,9 @@ const Modal: FunctionComponent<ComponentProps> = (
useEffect(() => {
const modal: HTMLDivElement | null = modalRef.current;
if (modal) {
// Focus the first focusable element in the modal
// Focus the first focusable element in the modal, excluding the close button
const focusableElements: NodeListOf<Element> = modal.querySelectorAll(
'button, [href], input, select, textarea, [tabindex]:not([tabindex="-1"])',
'button:not([data-testid="close-button"]), [href], input, select, textarea, [tabindex]:not([tabindex="-1"])',
);
const firstFocusable: HTMLElement | undefined = focusableElements[0] as
| HTMLElement