Fix #3721: Remove filterInputAutoFocus (#3723)

primefaces · Nov 30, 2022 · b56c806 · b56c806
1 parent f8d120d
commit b56c806
Show file tree

Hide file tree

Showing 5 changed files with 1 addition and 15 deletions.
diff --git a/api-generator/components/multiselect.js b/api-generator/components/multiselect.js
@@ -131,12 +131,6 @@ const MultiSelectProps = [
         default: 'true',
         description: 'When specified, displays an input field to filter the items on keyup.'
     },
-    {
-        name: 'filterInputAutoFocus',
-        type: 'boolean',
-        default: 'true',
-        description: 'When the panel is opened, it specifies that the filter input should focus automatically.'
-    },
     {
         name: 'filterBy',
         type: 'string',

diff --git a/components/doc/multiselect/apidoc.js b/components/doc/multiselect/apidoc.js
@@ -141,12 +141,6 @@ export function ApiDoc(props) {
                                 <td>true</td>
                                 <td>When specified, displays an input field to filter the items on keyup.</td>
                             </tr>
-                            <tr>
-                                <td>filterInputAutoFocus</td>
-                                <td>boolean</td>
-                                <td>true</td>
-                                <td>When the panel is opened, it specifies that the filter input should focus automatically.</td>
-                            </tr>
                             <tr>
                                 <td>filterBy</td>
                                 <td>string</td>

diff --git a/components/lib/multiselect/MultiSelect.js b/components/lib/multiselect/MultiSelect.js
@@ -675,7 +675,6 @@ MultiSelect.defaultProps = {
     emptyFilterMessage: null,
     filter: false,
     filterBy: null,
-    filterInputAutoFocus: true,
     filterLocale: undefined,
     filterMatchMode: 'contains',
     filterPlaceholder: null,

diff --git a/components/lib/multiselect/MultiSelectPanel.js b/components/lib/multiselect/MultiSelectPanel.js
@@ -26,7 +26,7 @@ export const MultiSelectPanel = React.memo(
 
         const onEntered = () => {
             props.onEntered(() => {
-                if (props.filter && props.filterInputAutoFocus) {
+                if (props.filter && filterInputRef.current) {
                     DomHandler.focus(filterInputRef.current, false);
                 }
             });

diff --git a/components/lib/multiselect/multiselect.d.ts b/components/lib/multiselect/multiselect.d.ts
@@ -85,7 +85,6 @@ export interface MultiSelectProps extends Omit<React.DetailedHTMLProps<React.Inp
     emptyFilterMessage?: MultiSelectEmptyFilterMessageType;
     filter?: boolean;
     filterBy?: string;
-    filterInputAutoFocus?: boolean;
     filterLocale?: string;
     filterMatchMode?: string;
     filterPlaceholder?: string;