fix: Remove tokenSeparators from showSearch

Author: 刘欢

README.md

@@ -85,6 +85,7 @@ export default () => (
 | dropdownAlign | additional align applied to dropdown | [AlignType](https://github.com/react-component/trigger/blob/728d7e92394aa4b3214650f743fc47e1382dfa68/src/interface.ts#L25-L80) | {} |
 | dropdownMenuStyle | additional style applied to dropdown menu | Object | React.CSSProperties |
 | notFoundContent | specify content to show when no result matches. | ReactNode | 'Not Found' |
+| tokenSeparators | separator used to tokenize on tag/multiple mode | string[]? |  |
 | open | control select open | boolean |  |
 | defaultOpen | control select default open | boolean |  |
 | placeholder | select placeholder | React Node |  |
@@ -140,11 +141,10 @@ export default () => (
 | name | description | type | default |
 | --- | --- | --- | --- |
 | autoClearSearchValue | auto clear search input value when multiple select is selected/deselected | boolean | true |
-| filterOption | whether filter options by input value. default filter by option's optionFilterProp prop's value | boolean | true/Function(inputValue:string, option:Option) |
+| filterOption | whether filter options by input value. default filter by option's optionFilterProp prop's value | boolean\| (inputValue: string, option: Option) => boolean | true |
 | filterSort | Sort function for search options sorting, see [Array.sort](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/sort)'s compareFunction. | Function(optionA:Option, optionB: Option) | - |
 | optionFilterProp | which prop value of option will be used for filter if filterOption is true | String | 'value' |
 | searchValue | The current input "search" text | string | - |
-| tokenSeparators | separator used to tokenize on tag/multiple mode | string[]? |  |
 | onSearch | called when input changed | function | - |
 
 ### Option props

src/Select.tsx

@@ -115,7 +115,6 @@ export interface SearchConfig<OptionType> {
   searchValue?: string;
   autoClearSearchValue?: boolean;
   onSearch?: (value: string) => void;
-  tokenSeparators?: string[];
   filterOption?: boolean | FilterFunc<OptionType>;
   filterSort?: (optionA: OptionType, optionB: OptionType, info: { searchValue: string }) => number;
   optionFilterProp?: string;
@@ -142,7 +141,6 @@ export interface SelectProps<ValueType = any, OptionType extends BaseOptionType
 
   /**  @deprecated pleace use  SearchConfig.onSearch */
   onSearch?: SearchConfig<OptionType>['onSearch'];
-  /**  @deprecated pleace use  SearchConfig.tokenSeparators */
   tokenSeparators?: string[];
   // >>> Options
   /**
@@ -234,7 +232,6 @@ const Select = React.forwardRef<BaseSelectRef, SelectProps<any, DefaultOptionTyp
       filterSort,
       onSearch,
       autoClearSearchValue = true,
-      tokenSeparators,
     } = searchConfig;
 
     const mergedId = useId(id);
@@ -724,7 +721,6 @@ const Select = React.forwardRef<BaseSelectRef, SelectProps<any, DefaultOptionTyp
           onSearch={onInternalSearch}
           autoClearSearchValue={autoClearSearchValue}
           onSearchSplit={onInternalSearchSplit}
-          tokenSeparators={tokenSeparators}
           popupMatchSelectWidth={popupMatchSelectWidth}
           // >>> OptionList
           OptionList={OptionList}

src/hooks/useSearchConfig.ts

@@ -7,7 +7,6 @@ const legacySearchProps = [
   'filterSort',
   'onSearch',
   'autoClearSearchValue',
-  'tokenSeparators',
 ];
 // Convert `showSearch` to unique config
 export default function useSearchConfig(
@@ -21,7 +20,6 @@ export default function useSearchConfig(
     filterSort,
     onSearch,
     autoClearSearchValue,
-    tokenSeparators,
   } = props || {};
   return React.useMemo<[boolean | undefined, SearchConfig<DefaultOptionType>]>(() => {
     const legacyShowSearch: SearchConfig<DefaultOptionType> = {};
@@ -46,6 +44,5 @@ export default function useSearchConfig(
     filterSort,
     onSearch,
     autoClearSearchValue,
-    tokenSeparators,
   ]);
 }

tests/Select.test.tsx

@@ -2630,14 +2630,14 @@ describe('Select.Basic', () => {
       expect(items[0].title).toBe('12');
       expect(items[2].title).toBe('12');
     });
-    it('autoClearSearchValue,tokenSeparators', () => {
+    it('autoClearSearchValue', () => {
       const { container } = render(
         <>
           <div id="test1">
-            <LegacyDemo showSearch autoClearSearchValue={false} tokenSeparators={[',', '*']} />
+            <LegacyDemo showSearch autoClearSearchValue={false} />
           </div>
           <div id="test2">
-            <CurrentDemo autoClearSearchValue={false} tokenSeparators={[',', '*']} />
+            <CurrentDemo autoClearSearchValue={false} />
           </div>
         </>,
       );
@@ -2652,10 +2652,6 @@ describe('Select.Basic', () => {
       fireEvent.click(items[1]);
       expect(legacyInput).toHaveValue('a');
       expect(currentInput).toHaveValue('a');
-      fireEvent.change(legacyInput, { target: { value: '1,' } });
-      fireEvent.change(currentInput, { target: { value: '1,' } });
-      expect(legacyInput).toHaveValue('');
-      expect(currentInput).toHaveValue('');
     });
   });
 });