diff --git a/.changeset/soft-pugs-travel.md b/.changeset/soft-pugs-travel.md new file mode 100644 index 0000000000..82edda38d4 --- /dev/null +++ b/.changeset/soft-pugs-travel.md @@ -0,0 +1,5 @@ +--- +"@nextui-org/dropdown": patch +--- + +fix unnecessary ref in dropdown (#4245) diff --git a/packages/components/dropdown/src/use-dropdown.ts b/packages/components/dropdown/src/use-dropdown.ts index fa78241ab3..9980c1a4e3 100644 --- a/packages/components/dropdown/src/use-dropdown.ts +++ b/packages/components/dropdown/src/use-dropdown.ts @@ -164,18 +164,12 @@ export function useDropdown(props: UseDropdownProps): UseDropdownReturn { }; }; - const getMenuTriggerProps: PropGetter = ( - originalProps = {}, - _ref: Ref | null | undefined = null, - ) => { + const getMenuTriggerProps: PropGetter = (originalProps = {}) => { // 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 {onPress, onPressStart, ...otherMenuTriggerProps} = menuTriggerProps; - return { - ...mergeProps(otherMenuTriggerProps, {isDisabled}, originalProps), - ref: mergeRefs(_ref, triggerRef), - }; + return mergeProps(otherMenuTriggerProps, {isDisabled}, originalProps); }; const getMenuProps = ( @@ -225,5 +219,5 @@ export type UseDropdownReturn = { disableAnimation: boolean; getPopoverProps: PropGetter; getMenuProps: (props?: Partial>, ref?: Ref) => MenuProps; - getMenuTriggerProps: (props?: any, ref?: Ref) => any; + getMenuTriggerProps: (props?: any) => any; };