-
-
Notifications
You must be signed in to change notification settings - Fork 46.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
refactor: timeline full token #35686
Conversation
Size Change: +418 B (0%) Total Size: 893 kB
|
Codecov Report
@@ Coverage Diff @@
## next #35686 +/- ##
==========================================
Coverage 100.00% 100.00%
==========================================
Files 469 472 +3
Lines 8490 8738 +248
Branches 2404 2507 +103
==========================================
+ Hits 8490 8738 +248
Continue to review full report at Codecov.
|
components/timeline/style/index.tsx
Outdated
timeLineItemPaddingBottom: number; | ||
timeLineItemHeadSize: number; | ||
timeLineItemCustomHeadPadding: string; | ||
timeLineRightPadding: number; |
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.
paddingInlineEnd
components/timeline/style/index.tsx
Outdated
const timeLineToken = mergeToken<TimelineToken>(token, { | ||
timeLineItemPaddingBottom: token.padding * 1.25, | ||
timeLineItemHeadSize: 10, | ||
timeLineItemCustomHeadPadding: '3px 1px', |
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.
这个数字有点问题,我看了一下现在的 timeline 这个是 18,但是右侧的是 22。应该和 22 对齐,用 fontSize * lineHeight 反推一个出来~
[中文版模板 / Chinese template]
🤔 This is a ...
🔗 Related issue link
💡 Background and solution
📝 Changelog
☑️ Self Check before Merge