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.

Sign up for GitHub

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

Jump to bottom

Modal.confirm能否支持maskClosable选项 #4488

Closed
pixystone opened this issue Jan 5, 2017 · 4 comments
Closed

Modal.confirm能否支持maskClosable选项 #4488

pixystone opened this issue Jan 5, 2017 · 4 comments
Labels
help wanted The suggestion or request has been accepted, we need you to help us by sending a pull request.

Comments

@pixystone
Copy link
Contributor

很多人都习惯去点蒙层
@afc163
Copy link
Member

afc163 commented Jan 5, 2017 

confirm({
  title: 'Want to delete these items?',
  content: 'some descriptions',
+ maskClosable: true,   // confirm 增加 maskClosable 配置，默认为 false
});

来个 PR 发到 feature-2.7 分支如何？

@afc163 afc163 added Feature help wanted The suggestion or request has been accepted, we need you to help us by sending a pull request. labels Jan 5, 2017
@pixystone
Copy link
Contributor Author

我试试

@afc163
Copy link
Member

afc163 commented Jan 5, 2017

赞，参与起来~ 后面用起来更有信心

pixystone added a commit to pixystone/ant-design that referenced this issue Jan 5, 2017
@pixystone
feat: Modal.confirm add maskClosable option (ant-design#4488)
ef00946
@pixystone pixystone mentioned this issue Jan 5, 2017
Merged
pixystone added a commit to pixystone/ant-design that referenced this issue Jan 6, 2017
@pixystone
feat: Modal.confirm add maskClosable option (ant-design#4488)
a547503
afc163 pushed a commit that referenced this issue Jan 7, 2017
@pixystone @afc163
feat: Modal.confirm add maskClosable option (#4488) (#4490)
b949d88 
* feat: Modal.confirm add maskClosable option (#4488)

* feat: Modal.confirm add maskClosable option (#4488)
@afc163 afc163 closed this as completed Jan 7, 2017
afc163 pushed a commit that referenced this issue Jan 8, 2017
@pixystone @afc163
feat: Modal.confirm add maskClosable option (#4488) (#4490)
a2bef2b 
* feat: Modal.confirm add maskClosable option (#4488)

* feat: Modal.confirm add maskClosable option (#4488)
@Alex1990 Alex1990 mentioned this issue Jul 6, 2017
Closed
10 tasks
@lock
Copy link

lock bot commented May 2, 2018

This thread has been automatically locked because it has not had recent activity. Please open a new issue for related bugs and link to relevant comments in this thread.

@lock lock bot locked as resolved and limited conversation to collaborators May 2, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
help wanted The suggestion or request has been accepted, we need you to help us by sending a pull request.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@afc163 @pixystone