fix(Dialog): panel type * 7

Author: tenphi

src/components/overlays/Dialog/Dialog.tsx

@@ -150,11 +150,7 @@ export const Dialog = forwardRef(function Dialog(
   }, [props, ref]);
 
   return (
-    <FocusScope
-      restoreFocus
-      autoFocus={isEntered}
-      contain={isEntered && context.type !== 'panel'}
-    >
+    <FocusScope restoreFocus contain={isEntered && context.type !== 'panel'}>
       {content}
     </FocusScope>
   );
@@ -202,11 +198,15 @@ const DialogContent = forwardRef(function DialogContent(
 
   // Focus the first focusable element in the dialog when it opens
   useEffect(() => {
-    setTimeout(() => {
-      focusManager?.focusFirst();
-      setIsCloseDisabled(false);
-    });
-  }, []);
+    if (contextProps.isOpen) {
+      setTimeout(() => {
+        focusManager?.focusFirst();
+        setIsCloseDisabled(false);
+      });
+    } else {
+      setIsCloseDisabled(true);
+    }
+  }, [contextProps.isOpen]);
 
   // let hasHeader = useHasChild('[data-id="Header"]', domRef);
   // let hasFooter = useHasChild('[data-id="Footer"]', domRef);

src/components/overlays/Dialog/DialogContainer.tsx

@@ -64,6 +64,7 @@ export function DialogContainer(props: CubeDialogContainerProps) {
     type,
     onClose: onDismiss,
     isDismissable,
+    isOpen,
   };
 
   return (

src/components/overlays/Dialog/DialogTrigger.tsx

@@ -309,6 +309,7 @@ function DialogTriggerBase(props) {
     type,
     onClose,
     isDismissable,
+    isOpen: state.isOpen,
     ...dialogProps,
   };
 

src/components/overlays/Dialog/context.tsx

@@ -11,6 +11,7 @@ export interface DialogContextValue extends HTMLAttributes<HTMLElement> {
     | 'panel';
   isDismissable?: boolean;
   onClose?: (arg?: string) => void;
+  isOpen?: boolean;
 }
 
 export const DialogContext = createContext<DialogContextValue | null>({});