fix(useVirtualList): get correct offset when itemHeight is fixed (#2279)

Co-authored-by: huangcheng <huangcheng.lq@bytedance.com>
alibaba · Apr 21, 2024 · 3ba7cb6 · 3ba7cb6
1 parent 6d3c3ba
commit 3ba7cb6
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/packages/hooks/src/useVirtualList/__tests__/index.test.ts b/packages/hooks/src/useVirtualList/__tests__/index.test.ts
@@ -61,6 +61,8 @@ describe('useVirtualList', () => {
 
       expect(hook.result.current[0].length).toBe(10);
       expect(container.scrollTop).toBe(20 * 30);
+      expect(hook.result.current[0][0].data).toBe(20);
+      expect(hook.result.current[0][0].index).toBe(20);
     });
 
     it('test with dynamic height', async () => {

diff --git a/packages/hooks/src/useVirtualList/index.ts b/packages/hooks/src/useVirtualList/index.ts
@@ -51,7 +51,7 @@ const useVirtualList = <T = any>(list: T[], options: Options<T>) => {
 
   const getOffset = (scrollTop: number) => {
     if (isNumber(itemHeightRef.current)) {
-      return Math.floor(scrollTop / itemHeightRef.current) + 1;
+      return Math.floor(scrollTop / itemHeightRef.current);
     }
     let sum = 0;
     let offset = 0;