-
-
Notifications
You must be signed in to change notification settings - Fork 46.7k
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
fix(style): keep checkbox.group style the same as v4 #42103
Conversation
components/checkbox/style/index.ts
Outdated
[`${checkboxCls}-group-item`]: { | ||
...resetComponent(token), | ||
|
||
marginRight: '8px', |
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.
marginRight: '8px', | |
marginInlineEnd: token.marginXS |
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## master #42103 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 617 617
Lines 10545 10544 -1
Branches 2883 2883
=========================================
- Hits 10545 10544 -1
☔ View full report in Codecov by Sentry. |
…o fix-checkbox-group
…o fix-checkbox-group
components/checkbox/style/index.ts
Outdated
@@ -36,6 +36,13 @@ export const genCheckboxStyle: GenerateStyle<CheckboxToken> = (token) => { | |||
...resetComponent(token), | |||
|
|||
display: 'inline-flex', | |||
flexWrap: 'wrap', | |||
columnGap: '8px', |
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.
用 token 哈
components/checkbox/style/index.ts
Outdated
columnGap: '8px', | ||
|
||
// Group > Grid | ||
'.ant-row': { |
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.
v4 没有这个样式吧
components/checkbox/style/index.ts
Outdated
'.ant-row': { | ||
width: '100%', | ||
}, |
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.
'.ant-row': { | |
width: '100%', | |
}, | |
[`> ${token.antCls}-row`]: { | |
flex: 1, | |
}, |
ant
前缀要用 token,这是动态的- flex 布局下用
flex: 1
撑满 - 用
>
只选择子代
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.
ok
[中文版模板 / Chinese template]
🤔 This is a ...
🔗 Related issue link
💡 Background and solution
📝 Changelog
☑️ Self-Check before Merge
🚀 Summary
🤖 Generated by Copilot at 1afca5d
Improved the styling of
checkbox
andcheckbox.Group
components to support flex and horizontal layouts. Modifiedcomponents/checkbox/style/index.ts
to adjust the display and margin properties.🔍 Walkthrough
🤖 Generated by Copilot at 1afca5d
display
toinline-block
and adding margin-right to checkbox group items (link)