Fix: fix scroll params (#728)

* Fix: fix scroll params

[issue-link](ant-design/ant-design#34468)

* Chore: add unit test for scroll config type params

Author: ManiacHanz

src/BaseSelect.tsx

@@ -4,7 +4,7 @@ import type { AlignType } from 'rc-trigger/lib/interface';
 import KeyCode from 'rc-util/lib/KeyCode';
 import isMobile from 'rc-util/lib/isMobile';
 import { useComposeRef } from 'rc-util/lib/ref';
-import type { ScrollTo } from 'rc-virtual-list/lib/List';
+import type { ScrollTo, ScrollConfig } from 'rc-virtual-list/lib/List';
 import useMergedState from 'rc-util/lib/hooks/useMergedState';
 import useLayoutEffect from 'rc-util/lib/hooks/useLayoutEffect';
 import { getSeparatedContent } from './utils/valueUtil';
@@ -46,7 +46,7 @@ export type RawValueType = string | number;
 export interface RefOptionListProps {
   onKeyDown: React.KeyboardEventHandler;
   onKeyUp: React.KeyboardEventHandler;
-  scrollTo?: (index: number) => void;
+  scrollTo?: (args: number | ScrollConfig) => void;
 }
 
 export type CustomTagProps = {
@@ -316,7 +316,7 @@ const BaseSelect = React.forwardRef((props: BaseSelectProps, ref: React.Ref<Base
   React.useImperativeHandle(ref, () => ({
     focus: selectorRef.current?.focus,
     blur: selectorRef.current?.blur,
-    scrollTo: (arg) => listRef.current?.scrollTo(arg as any),
+    scrollTo: (arg) => listRef.current?.scrollTo(arg),
   }));
 
   // ========================== Search Value ==========================

src/OptionList.tsx

@@ -1,5 +1,6 @@
 import * as React from 'react';
 import { useEffect } from 'react';
+import type { ScrollConfig } from 'rc-virtual-list/lib/List';
 import KeyCode from 'rc-util/lib/KeyCode';
 import omit from 'rc-util/lib/omit';
 import pickAttrs from 'rc-util/lib/pickAttrs';
@@ -20,7 +21,7 @@ export type OptionListProps = Record<string, never>;
 export interface RefOptionListProps {
   onKeyDown: React.KeyboardEventHandler;
   onKeyUp: React.KeyboardEventHandler;
-  scrollTo?: (index: number) => void;
+  scrollTo?: (args: number | ScrollConfig) => void;
 }
 
 function isTitleType(content: any) {
@@ -65,9 +66,9 @@ const OptionList: React.ForwardRefRenderFunction<RefOptionListProps, OptionListP
     event.preventDefault();
   };
 
-  const scrollIntoView = (index: number) => {
+  const scrollIntoView = (args: number | ScrollConfig) => {
     if (listRef.current) {
-      listRef.current.scrollTo({ index });
+      listRef.current.scrollTo(args);
     }
   };
 

tests/Select.test.tsx

@@ -3,6 +3,7 @@ import KeyCode from 'rc-util/lib/KeyCode';
 import React from 'react';
 import { act } from 'react-dom/test-utils';
 import { resetWarned } from 'rc-util/lib/warning';
+import type { ScrollConfig } from 'rc-virtual-list/lib/List';
 import { spyElementPrototype } from 'rc-util/lib/test/domHook';
 import VirtualList from 'rc-virtual-list';
 import type { SelectProps } from '../src';
@@ -1776,13 +1777,12 @@ describe('Select.Basic', () => {
     });
   });
 
-  it('scrollTo should work', () => {
+  it('scrollTo should work with number', () => {
     const ref = React.createRef<BaseSelectRef>();
     const wrapper = mount(<Select ref={ref} />);
 
     // Not crash
     ref.current.scrollTo(100);
-
     // Open to call again
     wrapper.setProps({
       open: true,
@@ -1791,6 +1791,24 @@ describe('Select.Basic', () => {
     ref.current.scrollTo(100);
   });
 
+  it('scrollTo should work with scrollConfig object', () => {
+    const ref = React.createRef<BaseSelectRef>();
+    const wrapper = mount(<Select ref={ref} />);
+    const scrollParams: ScrollConfig = {
+      index: 30,
+      align: 'top',
+    };
+
+    // Not crash
+    ref.current.scrollTo(scrollParams);
+    // Open to call again
+    wrapper.setProps({
+      open: true,
+    });
+    wrapper.update();
+    ref.current.scrollTo(scrollParams);
+  });
+
   it('pass props', () => {
     // `count` is not a valid dom prop. Just compatible with origin logic.
     const wrapper = mount(