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
support Modal.confirm({ icon: null })
to hide default Icon
#15319
Conversation
Deploy preview for ant-design ready! Built with commit 77c431b |
This pull request introduces 1 alert when merging dd44b0a6560f9bbe10953b63919be949bfc7d246 into b807859 - view on LGTM.com new alerts:
Comment posted by LGTM.com |
Codecov Report
@@ Coverage Diff @@
## feature #15319 +/- ##
========================================
Coverage 94.1% 94.1%
========================================
Files 250 250
Lines 6635 6635
Branches 1938 1938
========================================
Hits 6244 6244
Misses 390 390
Partials 1 1
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## feature #15319 +/- ##
===========================================
+ Coverage 94.1% 94.13% +0.03%
===========================================
Files 250 250
Lines 6635 6635
Branches 1938 1906 -32
===========================================
+ Hits 6244 6246 +2
+ Misses 390 388 -2
Partials 1 1
Continue to review full report at Codecov.
|
Mixed other commits, please clean up. |
First of all, thank you for your contribution! 😄
New feature please send pull request to feature branch, and rest to master branch.
Pull request will be merged after one of collaborators approve.
Please makes sure that these form are filled before submitting your pull request, thank you!
[中文版模板 / Chinese template]
🤔 This is a ...
👻 What's the background?
希望在调用
Modal.confirm
时可以省略图标💡 Solution
📝 Changelog description
No break changes.
☑️ Self Check before Merge