From be004fa4dab3c61b8396194c7604641ab2d82aad Mon Sep 17 00:00:00 2001
From: Xiao Yijun <xiaoyijun@silverhand.io>
Date: Tue, 5 Jul 2022 18:26:39 +0800
Subject: [PATCH] fix(console): leave page button should be primary on unsaved
 changes alert modal (#1421)

---
 packages/console/src/components/ConfirmModal/index.tsx |  4 +---
 .../src/components/UnsavedChangesAlertModal/index.tsx  | 10 ++++------
 2 files changed, 5 insertions(+), 9 deletions(-)

diff --git a/packages/console/src/components/ConfirmModal/index.tsx b/packages/console/src/components/ConfirmModal/index.tsx
index e58b9b37b..9384616ba 100644
--- a/packages/console/src/components/ConfirmModal/index.tsx
+++ b/packages/console/src/components/ConfirmModal/index.tsx
@@ -19,7 +19,6 @@ export type ConfirmModalProps = {
   isOpen: boolean;
   onCancel: () => void;
   onConfirm: () => void;
-  onClose?: () => void;
 };
 
 const ConfirmModal = ({
@@ -32,7 +31,6 @@ const ConfirmModal = ({
   isOpen,
   onCancel,
   onConfirm,
-  onClose = onCancel,
 }: ConfirmModalProps) => {
   return (
     <ReactModal
@@ -49,7 +47,7 @@ const ConfirmModal = ({
           </>
         }
         className={classNames(styles.content, className)}
-        onClose={onClose}
+        onClose={onCancel}
       >
         {children}
       </ModalLayout>
diff --git a/packages/console/src/components/UnsavedChangesAlertModal/index.tsx b/packages/console/src/components/UnsavedChangesAlertModal/index.tsx
index 428f2adc8..9a7d64371 100644
--- a/packages/console/src/components/UnsavedChangesAlertModal/index.tsx
+++ b/packages/console/src/components/UnsavedChangesAlertModal/index.tsx
@@ -68,12 +68,10 @@ const UnsavedChangesAlertModal = ({ hasUnsavedChanges }: Props) => {
   return (
     <ConfirmModal
       isOpen={displayAlert}
-      confirmButtonType="primary"
-      confirmButtonText="admin_console.general.stay_on_page"
-      cancelButtonText="admin_console.general.leave_page"
-      onCancel={leavePage}
-      onConfirm={stayOnPage}
-      onClose={stayOnPage}
+      confirmButtonText="admin_console.general.leave_page"
+      cancelButtonText="admin_console.general.stay_on_page"
+      onCancel={stayOnPage}
+      onConfirm={leavePage}
     >
       {t('general.unsaved_changes_warning')}
     </ConfirmModal>