Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dialog.confirm 需要能配置 closable #34

Open
fengsx opened this issue Jun 16, 2018 · 3 comments
Open

Dialog.confirm 需要能配置 closable #34

fengsx opened this issue Jun 16, 2018 · 3 comments

Comments

@fengsx
Copy link

fengsx commented Jun 16, 2018

目前直接写死了 closable

<Dialog
      prefixCls="kuma-dlg"
      className={`kuma-dlg-confirm ${newProps.className}`}
      onCancel={onCancel}
      visible
      closable
      title=""
      footer=""
      transitionName={newProps.transitionName}
      maskTransitionName={newProps.maskTransitionName}
      width={newProps.width}
      htmlClassName={newProps.htmlClassName}
      getContainer={newProps.getContainer}
    >
      <div>{body} {footer}</div>
    </Dialog>
@eternalsky
Copy link
Member

confirm 这种模式就是希望点击按钮的吧,如果是要 closable,为什么不直接使用标签形式的 Dialog?

@fengsx
Copy link
Author

fengsx commented Jun 19, 2018

模态的可以不用通过 visible 控制,直接类似 Dialog.confirm 的方式使用

@eternalsky
Copy link
Member

这个 owner 看一下吧,开下应该配置应该还好的。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants