refactor(dropdown): pass onKeyDown to menu trigger and don't hardcode…

… autoFocus
nextui-org · May 15, 2024 · 2de66ad · 2de66ad
1 parent 0bf8da5
commit 2de66ad
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/packages/components/dropdown/src/use-dropdown.ts b/packages/components/dropdown/src/use-dropdown.ts
@@ -126,7 +126,7 @@ export function useDropdown(props: UseDropdownProps) {
   ) => {
     // These props are not needed for the menu trigger since it is handled by the popover trigger.
     // eslint-disable-next-line @typescript-eslint/no-unused-vars
-    const {onKeyDown, onPress, onPressStart, ...otherMenuTriggerProps} = menuTriggerProps;
+    const {onPress, onPressStart, ...otherMenuTriggerProps} = menuTriggerProps;
 
     return {
       ...mergeProps(otherMenuTriggerProps, {isDisabled}, originalProps),
@@ -146,7 +146,7 @@ export function useDropdown(props: UseDropdownProps) {
         onAction: () => onMenuAction(props?.closeOnSelect),
         onClose: state.close,
       }),
-      autoFocus: "first",
+      autoFocus: state.focusStrategy || true,
     } as MenuProps;
   };