idux icon indicating copy to clipboard operation
idux copied to clipboard

fix(comp:table): scrollbar hides when not vertically overflowed

Open sallerli1 opened this issue 3 years ago • 3 comments

PR Checklist

Please check if your PR fulfills the following requirements:

  • [x] The commit message follows our guidelines
  • [x] Tests for the changes have been added/updated or not needed
  • [x] Docs and demo have been added/updated or not needed

What is the current behavior?

竖向内容不需要滚动时,仍然存在竖向滚动条

What is the new behavior?

修复以上问题

Other information

无论是否是虚拟滚动,只有在各自方向有内容溢出才展示滚动条

sallerli1 avatar Nov 01 '22 08:11 sallerli1

This preview will be available after the AzureCI is passed.

idux-bot[bot] avatar Nov 01 '22 08:11 idux-bot[bot]

Codecov Report

Merging #1246 (d8bd7ef) into main (99e4ddd) will decrease coverage by 0.14%. The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1246      +/-   ##
==========================================
- Coverage   93.05%   92.90%   -0.15%     
==========================================
  Files         320      320              
  Lines       29469    29469              
  Branches     3344     3337       -7     
==========================================
- Hits        27422    27379      -43     
- Misses       2047     2090      +43     
Impacted Files Coverage Δ
packages/cdk/scroll/src/utils.ts 52.63% <0.00%> (-28.29%) :arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

codecov[bot] avatar Nov 01 '22 08:11 codecov[bot]

@sallerli1 rebase 一下,然后解决一下 test

danranVm avatar Nov 02 '22 03:11 danranVm