-
Notifications
You must be signed in to change notification settings - Fork 269
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
feat(dialog): 增加关闭按钮 #1872
feat(dialog): 增加关闭按钮 #1872
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## next #1872 +/- ##
==========================================
+ Coverage 75.87% 75.92% +0.05%
==========================================
Files 215 215
Lines 8662 8673 +11
Branches 2480 2484 +4
==========================================
+ Hits 6572 6585 +13
+ Misses 1893 1891 -2
Partials 197 197 ☔ View full report in Codecov by Sentry. |
@@ -25,6 +26,8 @@ const defaultProps = { | |||
disableConfirmButton: false, | |||
footerDirection: 'horizontal', | |||
lockScroll: true, | |||
closeIconPosition: 'top-right', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
增加类型,明确可以使用哪些组合
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
增加类型,明确可以使用哪些组合
已修改
@@ -32,6 +33,8 @@ const defaultProps = { | |||
disableConfirmButton: false, | |||
footerDirection: 'horizontal', | |||
lockScroll: true, | |||
closeIconPosition: 'top-right', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里也更新下类型
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里也更新下类型
已修改
🤔 这个变动的性质是?
🔗 相关 Issue
💡 需求背景和解决方案
☑️ 请求合并前的自查清单