refactor(component): adjust confirm modal api (#7589)

This commit is contained in:
EYHN
2024-07-24 08:18:33 +00:00
parent 6ca7c41861
commit 98281a6394
28 changed files with 51 additions and 54 deletions
@@ -22,9 +22,7 @@ export const UsingHook = () => {
const showConfirm = () => {
openConfirmModal({
cancelText: 'Cancel',
confirmButtonOptions: {
children: 'Confirm',
},
confirmText: 'Confirm',
title: 'Confirm Modal',
children: 'Are you sure you want to confirm?',
onConfirm,
@@ -43,9 +41,7 @@ export const AutoClose = () => {
const onConfirm = () => {
openConfirmModal({
cancelText: 'Cancel',
confirmButtonOptions: {
children: 'Confirm',
},
confirmText: 'Confirm',
title: 'Confirm Modal',
children: 'Are you sure you want to confirm?',
onConfirm: () => console.log('Confirmed'),
@@ -10,11 +10,12 @@ import { Modal } from './modal';
import * as styles from './styles.css';
export interface ConfirmModalProps extends ModalProps {
confirmButtonOptions?: ButtonProps;
confirmButtonOptions?: Omit<ButtonProps, 'children'>;
onConfirm?: (() => void) | (() => Promise<void>);
onCancel?: () => void;
cancelText?: string;
cancelButtonOptions?: ButtonProps;
confirmText?: React.ReactNode;
cancelText?: React.ReactNode;
cancelButtonOptions?: Omit<ButtonProps, 'children'>;
reverseFooter?: boolean;
}
@@ -22,6 +23,7 @@ export const ConfirmModal = ({
children,
confirmButtonOptions,
// FIXME: we need i18n
confirmText,
cancelText = 'Cancel',
cancelButtonOptions,
reverseFooter,
@@ -64,7 +66,9 @@ export const ConfirmModal = ({
{cancelText}
</Button>
</DialogTrigger>
<Button onClick={onConfirmClick} {...confirmButtonOptions}></Button>
<Button onClick={onConfirmClick} {...confirmButtonOptions}>
{confirmText}
</Button>
</div>
</Modal>
);
@@ -58,10 +58,10 @@ const ConfirmModalTemplate: StoryFn<ConfirmModalProps> = () => {
onConfirm={handleConfirm}
title="Modal Title"
description="Modal description"
confirmText="Confirm"
confirmButtonOptions={{
loading: loading,
type: 'primary',
children: 'Confirm',
}}
>
<Input placeholder="input someting" status={inputStatus} />