From c1055463cce84c16e06f2eaea13ffe10c889aa6a Mon Sep 17 00:00:00 2001
From: Fernando Zanutto <fernandozanutto@outlook.com>
Date: Tue, 23 Apr 2024 19:11:38 -0300
Subject: [PATCH] feat: removing necessity of id on outer div

---
 src/renderer/components/modal/modal.tsx | 15 +++++++--------
 1 file changed, 7 insertions(+), 8 deletions(-)

diff --git a/src/renderer/components/modal/modal.tsx b/src/renderer/components/modal/modal.tsx
index 2b31998b..a1cfb087 100644
--- a/src/renderer/components/modal/modal.tsx
+++ b/src/renderer/components/modal/modal.tsx
@@ -41,10 +41,10 @@ export function Modal({
   };
 
   const isTopMostModal = () => {
-    const openModals = document.querySelectorAll("[role=modal-container]");
+    const openModals = document.querySelectorAll("[role=modal-content]");
     return (
       openModals.length &&
-      openModals[openModals.length - 1].id === "modal-container-" + componentId
+      openModals[openModals.length - 1] === modalContentRef.current
     );
   };
 
@@ -63,7 +63,9 @@ export function Modal({
     const onMouseDown = (e: MouseEvent) => {
       if (!isTopMostModal()) return;
 
-      const clickedOutsideContent = !modalContentRef.current.contains(e.target as Node);
+      const clickedOutsideContent = !modalContentRef.current.contains(
+        e.target as Node
+      );
 
       if (clickedOutsideContent) {
         handleCloseClick();
@@ -81,13 +83,10 @@ export function Modal({
   if (!visible) return null;
 
   return createPortal(
-    <div
-      className={styles.backdrop({ closing: isClosing })}
-      role="modal-container"
-      id={"modal-container-" + componentId}
-    >
+    <div className={styles.backdrop({ closing: isClosing })}>
       <div
         className={styles.modal({ closing: isClosing })}
+        role="modal-content"
         ref={modalContentRef}
       >
         <div className={styles.modalHeader}>