-
-
Notifications
You must be signed in to change notification settings - Fork 50.5k
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(Table): Hide header extra scrollbar control #16950
Conversation
3dc8655
to
bb0314c
Compare
bb0314c
to
f1c43c1
Compare
Codecov Report
@@ Coverage Diff @@
## master #16950 +/- ##
==========================================
- Coverage 96% 95.96% -0.05%
==========================================
Files 263 263
Lines 7335 7335
Branches 2007 2046 +39
==========================================
- Hits 7042 7039 -3
- Misses 291 294 +3
Partials 2 2
Continue to review full report at Codecov.
|
@yoyo837 有重现么 |
@afc163 我上 |
https://ant.design/components/table-cn/#components-table-demo-fixed-header 的 demo 加个 bordered 就出来了 |
如果全部加上width,则错位 |
@qq645381995 给在线演示 |
@afc163 做了半天排除法,确定了是重置样式 @line-height-base : 1.6; 改成了 1.6 的原因,这个是因为什么问题呢? |
🤔 This is a ...
👻 What's the background?
close #4637
close #14211
close #10796
在 #4637 (comment) 里尝试过四种方法解决这个问题,然而当时都引起了其他问题,于是放弃了修复。
💡 Solution
2018 年有人回复方法四可以解决(实际上还是不行,会引发 #4936),继续这个方案,依据 https://stackoverflow.com/a/23200767 更换为
background-color: transparent
,并根据 https://stackoverflow.com/a/54101063 解决了 Firefox 下的问题。这个方案依旧会引起的 #4936 问题,原因是给滚动条设置背景色(或修改透明度)会影响滚动条宽度的计算,在 react-component/table#333 里进行了针对性处理。
📝 Changelog
☑️ Self Check before Merge