fix: apply rawOpen fix to multiple mode (#1202)

* fix: apply rawOpen fix to multiple mode

Pass rawOpen through context and use it in MultipleContent to
prevent search value from being cleared when emptyListContent
blocks the dropdown from opening. Also adds test coverage
for the multiple mode case.

Co-Authored-By: Claude Opus 4.5 <[email protected]>

* test: refactor typing test to use it.each

Combine single and multiple mode tests into a parametrized
test using it.each for better maintainability.

Co-Authored-By: Claude Opus 4.5 <[email protected]>

---------

Co-authored-by: Claude Opus 4.5 <[email protected]>

Author: zombieJ

src/BaseSelect/index.tsx

@@ -645,6 +645,7 @@ const BaseSelect = React.forwardRef<BaseSelectRef, BaseSelectProps>((props, ref)
       notFoundContent,
       open: mergedOpen,
       triggerOpen: mergedOpen,
+      rawOpen,
       id,
       showSearch,
       multiple,
@@ -662,6 +663,7 @@ const BaseSelect = React.forwardRef<BaseSelectRef, BaseSelectProps>((props, ref)
       showSearch,
       multiple,
       mergedOpen,
+      rawOpen,
       showScrollBar,
       styles,
       classNames,

src/SelectInput/Content/MultipleContent.tsx

@@ -36,6 +36,7 @@ export default React.forwardRef<HTMLInputElement, SharedContentProps>(function M
     disabled,
     showSearch,
     triggerOpen,
+    rawOpen,
     toggleOpen,
     autoClearSearchValue,
     tagRender: tagRenderFromContext,
@@ -50,8 +51,9 @@ export default React.forwardRef<HTMLInputElement, SharedContentProps>(function M
 
   // ===================== Search ======================
   // Apply autoClearSearchValue logic: when dropdown is closed and autoClearSearchValue is not false (default true), clear search value
+  // Use rawOpen to avoid clearing search when emptyListContent blocks open
   let computedSearchValue = searchValue;
-  if (!triggerOpen && mode === 'multiple' && autoClearSearchValue !== false) {
+  if (!rawOpen && mode === 'multiple' && autoClearSearchValue !== false) {
     computedSearchValue = '';
   }
 

src/hooks/useBaseProps.ts

@@ -8,6 +8,7 @@ import type { BaseSelectProps } from '../BaseSelect';
 
 export interface BaseSelectContextProps extends BaseSelectProps {
   triggerOpen: boolean;
+  rawOpen: boolean;
   multiple: boolean;
   toggleOpen: (open?: boolean) => void;
   lockOptions: boolean;

tests/Select.test.tsx

@@ -1997,32 +1997,37 @@ describe('Select.Basic', () => {
     expect(container.querySelector('.rc-select-dropdown-empty')).toBeFalsy();
   });
 
-  it('should allow typing when notFoundContent is null and no options match', () => {
-    const onSearch = jest.fn();
-    const { container } = render(
-      <Select showSearch notFoundContent={null} onSearch={onSearch}>
-        <Option value="jack">Jack</Option>
-        <Option value="lucy">Lucy</Option>
-      </Select>,
-    );
+  describe('should allow typing when notFoundContent is null and no options match', () => {
+    it.each<{ mode: 'multiple' | undefined; label: string }>([
+      { mode: undefined, label: 'single' },
+      { mode: 'multiple', label: 'multiple' },
+    ])('$label', ({ mode }) => {
+      const onSearch = jest.fn();
+      const { container } = render(
+        <Select mode={mode} showSearch notFoundContent={null} onSearch={onSearch}>
+          <Option value="jack">Jack</Option>
+          <Option value="lucy">Lucy</Option>
+        </Select>,
+      );
 
-    const input = container.querySelector('input');
+      const input = container.querySelector('input');
+
+      // Type 'j' - should match 'Jack'
+      fireEvent.change(input, { target: { value: 'j' } });
+      expect(onSearch).toHaveBeenLastCalledWith('j');
+      expect(input.value).toBe('j');
+      expect(container.querySelectorAll('.rc-select-item-option')).toHaveLength(1);
 
-    // Type 'j' - should match 'Jack'
-    fireEvent.change(input, { target: { value: 'j' } });
-    expect(onSearch).toHaveBeenLastCalledWith('j');
-    expect(input.value).toBe('j');
-    expect(container.querySelectorAll('.rc-select-item-option')).toHaveLength(1);
-
-    // Type 'x' - no match, but input should still work
-    fireEvent.change(input, { target: { value: 'x' } });
-    expect(onSearch).toHaveBeenLastCalledWith('x');
-    expect(input.value).toBe('x');
-
-    // Type more characters - should continue working
-    fireEvent.change(input, { target: { value: 'xyz' } });
-    expect(onSearch).toHaveBeenLastCalledWith('xyz');
-    expect(input.value).toBe('xyz');
+      // Type 'x' - no match, but input should still work
+      fireEvent.change(input, { target: { value: 'x' } });
+      expect(onSearch).toHaveBeenLastCalledWith('x');
+      expect(input.value).toBe('x');
+
+      // Type more characters - should continue working
+      fireEvent.change(input, { target: { value: 'xyz' } });
+      expect(onSearch).toHaveBeenLastCalledWith('xyz');
+      expect(input.value).toBe('xyz');
+    });
   });
 
   it('click outside to close select', () => {