Infrastructure/logto
0
Fork 0
mirror of https://github.com/logto-io/logto.git synced 2025-02-03 21:48:55 -05:00
Code Issues Activity

fix(console): leave page button should be primary on unsaved changes alert modal (#1421)

Browse source
This commit is contained in:
Xiao Yijun 2022-07-05 18:26:39 +08:00 • committed by GitHub
parent 1d5f69db12
commit be004fa4da
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 5 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
packages/console/src/components/ConfirmModal/index.tsx
View file

@ -19,7 +19,6 @@ export type ConfirmModalProps = {
isOpen: boolean; isOpen: boolean;
onCancel: () => void; onCancel: () => void;
onConfirm: () => void; onConfirm: () => void;
onClose?: () => void;
}; };
const ConfirmModal = ({ const ConfirmModal = ({
@ -32,7 +31,6 @@ const ConfirmModal = ({
isOpen, isOpen,
onCancel, onCancel,
onConfirm, onConfirm,
onClose = onCancel,
}: ConfirmModalProps) => { }: ConfirmModalProps) => {
return ( return (
<ReactModal <ReactModal
@ -49,7 +47,7 @@ const ConfirmModal = ({
</> </>
} }
className={classNames(styles.content, className)} className={classNames(styles.content, className)}
onClose={onClose} onClose={onCancel}
> >
{children} {children}
</ModalLayout> </ModalLayout>

10
packages/console/src/components/UnsavedChangesAlertModal/index.tsx
View file

@ -68,12 +68,10 @@ const UnsavedChangesAlertModal = ({ hasUnsavedChanges }: Props) => {
return ( return (
<ConfirmModal <ConfirmModal
isOpen={displayAlert} isOpen={displayAlert}
confirmButtonType="primary" confirmButtonText="admin_console.general.leave_page"
confirmButtonText="admin_console.general.stay_on_page" cancelButtonText="admin_console.general.stay_on_page"
cancelButtonText="admin_console.general.leave_page" onCancel={stayOnPage}
onCancel={leavePage} onConfirm={leavePage}
onConfirm={stayOnPage}
onClose={stayOnPage}
> >
{t('general.unsaved_changes_warning')} {t('general.unsaved_changes_warning')}
</ConfirmModal> </ConfirmModal>