-
-
Notifications
You must be signed in to change notification settings - Fork 53.5k
feat: ConfigProvider support classNames and styles for Result #52171
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
Conversation
|
👁 Visual Regression Report for PR #52171 Passed ✅
🎊 Congrats! No visual-regression diff found. |
WalkthroughThis pull request introduces a new feature to the Changes
|
@@ -52,14 +68,13 @@ const ExceptionStatus = Object.keys(ExceptionMap); | |||
*/ | |||
|
|||
interface IconProps { |
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.
The IconProps
interface has been modified to replace prefixCls
with className
and added an optional style
property. Ensure that all usages of IconProps
are updated accordingly to prevent potential runtime errors.
More templates
commit: |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## next #52171 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 763 763
Lines 13643 13650 +7
Branches 3561 3563 +2
=========================================
+ Hits 13643 13650 +7 ☔ View full report in Codecov by Sentry. |
components/result/demo/_semantic.tsx
Outdated
subTitle: '副标题元素', | ||
body: '内容元素', | ||
extra: '额外元素', | ||
image: '图标元素', |
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.
这个应该是 icon?
Bundle ReportChanges will increase total bundle size by 5.08kB (0.15%) ⬆️. This is within the configured threshold ✅ Detailed changes
|
components/result/index.tsx
Outdated
); | ||
|
||
const bodyClassNames = classNames( | ||
`${prefixCls}-content`, |
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.
这里也要变成 body,否则会让人疑惑的。挪到 next 里吧。
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.
done
9da6cb2
to
93f0955
Compare
93f0955
to
fb8089d
Compare
); | ||
|
||
const bodyClassNames = classNames( | ||
`${prefixCls}-body`, |
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.
这边替换后,style/index.ts
里的没有替换。
中文版模板 / Chinese template
🤔 This is a ...
🔗 Related Issues
💡 Background and Solution
📝 Change Log