-
-
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: form cssinjs #34679
refactor: form cssinjs #34679
Conversation
Some components are broken in Form.Item because |
Size Change: -2.9 kB (0%) Total Size: 929 kB
|
Codecov Report
@@ Coverage Diff @@
## next #34679 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 438 438
Lines 8107 8115 +8
Branches 2302 2301 -1
=========================================
+ Hits 8107 8115 +8
Continue to review full report at Codecov.
|
@@ -5188,7 +5188,7 @@ exports[`renders ./components/form/demo/register.md extend context correctly 1`] | |||
class="ant-form-item-control-input-content" | |||
> | |||
<div | |||
class="ant-input-textarea ant-input-textarea-show-count" | |||
class="ant-input-textarea ant-input-textarea-show-count ant-input-textarea-in-form-item" |
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.
这个应该是在 feature 上的?怎么 snapshot 也变了?
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.
这个样式没写在 components.less 里,漏掉了,我去 feature 上补一下
components/input-number/index.tsx
Outdated
@@ -55,7 +55,7 @@ const InputNumber = React.forwardRef<HTMLInputElement, InputNumberProps>((props, | |||
const prefixCls = getPrefixCls('input-number', customizePrefixCls); | |||
|
|||
// Style | |||
const [wrapSSR, hashId] = useStyle(prefixCls, iconPrefixCls); | |||
const [wrapSSR, hashId] = useStyle(prefixCls, iconPrefixCls || ''); |
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.
感觉有问题,这个 iconPrefixCls 应该总是存在才对
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.
这个 iconPrefixCls 不知道什么时候类型变成 string | undefined
了
components/form/style/index.tsx
Outdated
// Adjust output element | ||
output: { | ||
display: 'block', | ||
paddingTop: '15px', |
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.
number
components/form/style/index.tsx
Outdated
input[type='checkbox']:focus`]: { | ||
// outline: 'thin dotted', | ||
outline: '5px auto -webkit-focus-ring-color', | ||
outlineOffset: -2, |
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.
和标准 Input 一样用 token
components/form/style/index.tsx
Outdated
marginBottom: 0, | ||
|
||
'&-with-help': { | ||
marginBottom: 24, // FIXME: magic |
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.
有没有发现 24 反复出现,可以抽一个变量出来
[中文版模板 / Chinese template]
🤔 This is a ...
🔗 Related issue link
💡 Background and solution
📝 Changelog
☑️ Self Check before Merge