diff --git a/src/components/dialog/__tests__/__snapshots__/actionbutton.test.tsx.snap b/src/components/dialog/__tests__/__snapshots__/actionbutton.test.tsx.snap index 3d3df8812..165dd1592 100644 --- a/src/components/dialog/__tests__/__snapshots__/actionbutton.test.tsx.snap +++ b/src/components/dialog/__tests__/__snapshots__/actionbutton.test.tsx.snap @@ -3,7 +3,6 @@ exports[`Test ActionButton Component Match ActionButton Snapshot 1`] = ` ); diff --git a/src/components/dialog/confirm.tsx b/src/components/dialog/confirm.tsx index 5a4757ca5..52f5d61db 100644 --- a/src/components/dialog/confirm.tsx +++ b/src/components/dialog/confirm.tsx @@ -18,7 +18,7 @@ export default function confirm(config: IModalFuncProps) { const div = document.createElement('div'); document.body.appendChild(div); // eslint-disable-next-line @typescript-eslint/no-use-before-define - let currentConfig = { ...config, close, visible: true } as any; + const currentConfig = { ...config, close, visible: true } as any; function destroy(...args: any[]) { const triggerCancel = args.some( @@ -46,21 +46,16 @@ export default function confirm(config: IModalFuncProps) { function render({ okText, cancelText, ...props }: any) { renderUtils( , div ); } function close(...args: any[]) { - currentConfig = { - ...currentConfig, - visible: false, - afterClose: () => destroy(...args), - }; - render(currentConfig); + destroy(...args); } render(currentConfig); diff --git a/src/components/dialog/confirmDialog.tsx b/src/components/dialog/confirmDialog.tsx index 34eae9c47..c344f4701 100644 --- a/src/components/dialog/confirmDialog.tsx +++ b/src/components/dialog/confirmDialog.tsx @@ -29,14 +29,14 @@ const ConfirmDialog = (props: ConfirmDialogProps) => { onOk, close, maskStyle, - okText = 'delete', + okText = 'Ok', okButtonProps, - cancelText = 'cancel', + cancelText = 'Cancel', cancelButtonProps, + okCancel, bodyStyle, - closable = true, + closable = false, className, - okCancel, width = 520, style = {}, mask = true, @@ -44,11 +44,10 @@ const ConfirmDialog = (props: ConfirmDialogProps) => { transitionName = 'zoom', maskTransitionName = 'fade', type, - ...resetProps + visible, } = props; const confirmDescriperClassName = iconConfirmClassName(type); - const classString = classNames( confirmClassName, confirmDescriperClassName, @@ -58,8 +57,8 @@ const ConfirmDialog = (props: ConfirmDialogProps) => { const cancelButton = okCancel && ( {cancelText} @@ -73,16 +72,17 @@ const ConfirmDialog = (props: ConfirmDialogProps) => { [centeredConfirmClassName]: !!props.centered, })} onCancel={() => close({ triggerCancel: true })} - title="" transitionName={transitionName} - footer="" maskTransitionName={maskTransitionName} mask={mask} maskClosable={maskClosable} style={style} width={width} closable={closable} - {...resetProps} + footer="" + title="" + maskStyle={maskStyle} + visible={visible} >
@@ -105,8 +105,8 @@ const ConfirmDialog = (props: ConfirmDialogProps) => { { {okText} diff --git a/src/components/dialog/modal.tsx b/src/components/dialog/modal.tsx index 087bb3cad..d6a1e48fc 100644 --- a/src/components/dialog/modal.tsx +++ b/src/components/dialog/modal.tsx @@ -89,8 +89,8 @@ export const Modal: React.FC = (props: IModalProps) => { centered, getContainer, closeIcon, - cancelText = 'cancel', - okText = 'ok', + cancelText = 'Cancel', + okText = 'Ok', ...restProps } = props; diff --git a/src/components/dialog/style.scss b/src/components/dialog/style.scss index 0b2502087..b28e1a4e2 100644 --- a/src/components/dialog/style.scss +++ b/src/components/dialog/style.scss @@ -67,12 +67,14 @@ } &-footer { + border-radius: 0 0 2px 2px; display: flex; - font-size: 13px; + flex-direction: row; justify-content: flex-end; - padding: 20px 10px 10px; + padding: 10px 16px; - .mo-botton { + .mo-btn { + margin-left: 8px; width: fit-content; } } @@ -98,10 +100,10 @@ &--x { display: block; font-size: 16px; - height: 40px; + height: 54px; line-height: 40px; text-align: center; - width: 40px; + width: 54px; } } } @@ -155,7 +157,13 @@ &__btns { display: flex; + flex-direction: row; justify-content: flex-end; - padding: 20px 10px 10px; + padding: 10px 16px; + + .mo-btn { + margin-left: 8px; + width: fit-content; + } } } diff --git a/stories/components/17-Dialog.stories.tsx b/stories/components/17-Dialog.stories.tsx index ce167fa91..d5d9d6e80 100644 --- a/stories/components/17-Dialog.stories.tsx +++ b/stories/components/17-Dialog.stories.tsx @@ -24,14 +24,19 @@ stories.add('Basic Usage', () => { function showConfirm() { confirm({ - title: 'Are you sure you want to permanently delete ?', - content: 'This action is irreversible!', - cancelButtonProps: { disabled: true }, + title: 'Tweet us your feedback', + content: ( +
+

Some contents...

+

Some contents...

+

Some contents...

+
+ ), onOk() { - console.log('OK'); + console.log('onOk'); }, onCancel() { - console.log('Cancel'); + console.log('onCancel'); }, }); }