fix: adjust inVirtual logic(#227)

[NameWjp]

复现步骤：

1. 修改 examples 中 height 例子，将 data 数据调整为 10 条，目前能够滚动，但是没有滚动条
2. 调整逻辑后滚动条正常显示

相关 issue：https://github.com/ant-design/ant-design/issues/45935

[vercel[bot]]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
virtual-list	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Apr 5, 2024 4:59pm

[beautiful-boyyy]

是不是你antd版本太低了，最新版本没这问题了

[NameWjp]

@beautiful-boyyy 不是，你按照我的步骤是可以复现的，只是这个问题出现的概率较小（实际上就是你传入的：最小高度 * 数量 < 容器高度，但是 item 每一项的实际高度相加后 > 容器高度时，就会出现这个问题），最新版本的 antd 的虚拟滚动都有这个问题

[beautiful-boyyy]

@beautiful-boyyy 不是，你按照我的步骤是可以复现的，只是这个问题出现的概率较小（实际上就是你传入的：最小高度 * 数量 < 容器高度，但是 item 每一项的实际高度相加后 > 容器高度时，就会出现这个问题），最新版本的 antd 的虚拟滚动都有这个问题

提供一个codesandbox demo吧

[NameWjp]

@beautiful-boyyy 不是，你按照我的步骤是可以复现的，只是这个问题出现的概率较小（实际上就是你传入的：最小高度 * 数量 < 容器高度，但是 item 每一项的实际高度相加后 > 容器高度时，就会出现这个问题），最新版本的 antd 的虚拟滚动都有这个问题

提供一个codesandbox demo吧

codesandbox

[beautiful-boyyy]

@beautiful-boyyy 不是，你按照我的步骤是可以复现的，只是这个问题出现的概率较小（实际上就是你传入的：最小高度 * 数量 < 容器高度，但是 item 每一项的实际高度相加后 > 容器高度时，就会出现这个问题），最新版本的 antd 的虚拟滚动都有这个问题

提供一个codesandbox demo吧

codesandbox

貌似没权限

[NameWjp]

@beautiful-boyyy 不是，你按照我的步骤是可以复现的，只是这个问题出现的概率较小（实际上就是你传入的：最小高度 * 数量 < 容器高度，但是 item 每一项的实际高度相加后 > 容器高度时，就会出现这个问题），最新版本的 antd 的虚拟滚动都有这个问题

提供一个codesandbox demo吧

codesandbox

貌似没权限

忘记开了，再试下

[beautiful-boyyy]

@beautiful-boyyy 不是，你按照我的步骤是可以复现的，只是这个问题出现的概率较小（实际上就是你传入的：最小高度 * 数量 < 容器高度，但是 item 每一项的实际高度相加后 > 容器高度时，就会出现这个问题），最新版本的 antd 的虚拟滚动都有这个问题

提供一个codesandbox demo吧

codesandbox

你提供的demo没有用到antd List，antd List没有暴露 height，itemHeight接口，之前的bug也在前几个版本就修复了，virtual-list这里的逻辑确实有问题，单独使用的话确实会有bug，我之前也提了一个pr，但是测试没跑过，你可能需要提供正确的测试用例。

[NameWjp]

@zombieJ @yoyo837 这个 bug 可以看一下吗，需要咋调整下？

[yoyo837]

用例能覆盖到吗？

[NameWjp]

用例能覆盖到吗？

测试已经加上了，本地跑的没啥问题

[codecov[bot]]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 98.24%. Comparing base (e8e9aeb) to head (306dfcd).

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #262   +/-   ##
=======================================
  Coverage   98.23%   98.24%           
=======================================
  Files          18       18           
  Lines         681      684    +3     
  Branches      160      163    +3     
=======================================
+ Hits          669      672    +3     
  Misses         12       12           

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

[NameWjp]

@yoyo837 麻烦问下有发版计划吗，有个项目用的这个包，想修复这个问题

[yoyo837]

已知会Owner.

[yoyo837]

@yoyo837 麻烦问下有发版计划吗，有个项目用的这个包，想修复这个问题

https://github.com/react-component/virtual-list/releases/tag/v3.11.5

[NameWjp]

@yoyo837 麻烦问下有发版计划吗，有个项目用的这个包，想修复这个问题

https://github.com/react-component/virtual-list/releases/tag/v3.11.5

ok