-
-
Notifications
You must be signed in to change notification settings - Fork 49.8k
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: Steps support inline #38311
feat: Steps support inline #38311
Conversation
03aed33
to
a55a31e
Compare
Codecov ReportBase: 100.00% // Head: 100.00% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## next #38311 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 543 544 +1
Lines 9338 9348 +10
Branches 2636 2639 +3
=========================================
+ Hits 9338 9348 +10
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
a55a31e
to
d1b302a
Compare
components/steps/style/inline.ts
Outdated
'&-wait': { | ||
[`${componentCls}-item-icon ${componentCls}-icon ${componentCls}-icon-dot`]: { | ||
backgroundColor: token.colorBorderBg, | ||
border: `${token.lineWidth}px solid ${token.colorBorderSecondary}`, |
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.
border: `${token.lineWidth}px solid ${token.colorBorderSecondary}`, | |
border: `${token.lineWidth}px ${token.lineType} ${token.colorSplit}`, |
components/steps/style/inline.ts
Outdated
}, | ||
'&-finish': { | ||
[`${componentCls}-item-tail::after`]: { | ||
backgroundColor: token.colorBorderSecondary, |
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.
和 line 相同的话可以搞个变量共用下
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.
优先用 colorSplit
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.
colorSplit 有透明度会重叠,加个变量
Demo 里可以改一下状态和进度,现在都是一样的 |
201b3fb
to
68bb268
Compare
0cc942e
to
6ae39bc
Compare
有冲突了,看设计应该不会有大变化了,解一下合了吧 |
* test: update snapshot * docs: add inline demo * chore: upgrage rc-steps
6ae39bc
to
091135b
Compare
done |
hover 背景色加个 transition 过渡。 |
这个样式要加在普通状态上,放在 hover 上才加上 transition 会有渲染时机问题。 |
好哒~ |
[中文版模板 / Chinese template]
🤔 This is a ...
🔗 Related issue link
💡 Background and solution
预览效果:https://preview-38311-ant-design.surge.sh/components/steps-cn/#components-steps-demo-inline
📝 Changelog
☑️ Self-Check before Merge