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: optimize Input presuffix style(#17414) #17508
Conversation
Deploy preview for ant-design ready! Built with commit 9691d2b |
Codecov Report
@@ Coverage Diff @@
## master #17508 +/- ##
=======================================
Coverage 96.03% 96.03%
=======================================
Files 267 267
Lines 7419 7419
Branches 2029 2029
=======================================
Hits 7125 7125
Misses 292 292
Partials 2 2 Continue to review full report at Codecov.
|
1 similar comment
Codecov Report
@@ Coverage Diff @@
## master #17508 +/- ##
=======================================
Coverage 96.03% 96.03%
=======================================
Files 267 267
Lines 7419 7419
Branches 2029 2029
=======================================
Hits 7125 7125
Misses 292 292
Partials 2 2 Continue to review full report at Codecov.
|
@@ -373,9 +373,17 @@ | |||
color: @input-color; | |||
line-height: 0; | |||
transform: translateY(-50%); | |||
background-color: #fff; | |||
height: calc(100% - 2px); |
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.
不需要设 backgroud-color 的吧,还有 height 也有问题
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.
有多个suffix时.ant-input-suffix
覆盖在input上面,设置background-color和height是想用.ant-input-suffix
覆盖住input输入框。实际上这里判断suffix的数量来修改padding-right应该是更优的方案。
height这里应该把padding减去?
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.
应该用 padding,#fff
太粗暴了……
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.
@zombieJ 请老朱背锅😂
@LilyWakana 对的,之前的修复方法比较 hack,按理不应该合进去的。根据后者的思路,能否帮忙提下 PR? |
@LilyWakana |
其他问题: #17414 (comment) |
好的。 |
这个pr好像不能reopen了,重新提了一个 #17684 |
🤔 This is a ...
👻 What's the background?
close #17414
💡 Solution
📝 Changelog
☑️ Self Check before Merge