-
-
Notifications
You must be signed in to change notification settings - Fork 50.1k
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: Modal.config rootPrefixCls #25613
Conversation
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit 043e12a:
|
Codecov Report
@@ Coverage Diff @@
## feature #25613 +/- ##
===========================================
- Coverage 99.51% 99.51% -0.01%
===========================================
Files 372 372
Lines 7238 7237 -1
Branches 2024 1977 -47
===========================================
- Hits 7203 7202 -1
Misses 35 35
Continue to review full report at Codecov.
|
components/modal/ActionButton.tsx
Outdated
@@ -7,6 +7,7 @@ export interface ActionButtonProps { | |||
actionFn?: (...args: any[]) => any | PromiseLike<any>; | |||
closeModal: Function; | |||
autoFocus?: boolean; | |||
prefixCls?: string; |
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.
从改动部分看起来这个是一定要有的
[中文版模板 / Chinese template]
🤔 This is a ...
🔗 Related issue link
#25603 没能解决 #25586 (comment) 的问题。
💡 Background and solution
prefixCls
不够用,因为 Modal.confirm 里面还有按钮,因此不支持prefixCls
了,直接改成rootPrefixCls
。📝 Changelog
Modal.config
to setrootPrefixCls
for Modal statis methods.Modal.config
用于全局配置 Modal 静态方法的rootPrefixCls
.☑️ Self Check before Merge
View rendered components/modal/index.en-US.md
View rendered components/modal/index.zh-CN.md
View rendered docs/react/faq.en-US.md
View rendered docs/react/faq.zh-CN.md