解决shadcn-ui项目中Sidebar与DropdownMenu组件集成时的ref传递问题

2025-04-29 07:35:42作者：凤尚柏Louis

在React组件开发中，我们经常会遇到需要将不同组件库中的元素进行组合使用的情况。本文将以shadcn-ui项目为例，深入分析一个典型问题：当尝试在Sidebar组件中使用DropdownMenu组件时出现的ref传递错误，并提供专业级的解决方案。

问题背景

在shadcn-ui的组件生态中，Sidebar和DropdownMenu是两个常用的UI组件。开发者经常需要将它们组合使用，比如在侧边栏菜单项中添加下拉操作菜单。然而，当尝试将SidebarMenuAction组件作为DropdownMenuTrigger的触发元素时，React会抛出"Function components cannot be given refs"的错误提示。

这个问题的本质在于React的ref传递机制。DropdownMenuTrigger组件需要向其子组件传递ref属性，以便管理焦点和交互状态，但SidebarMenuAction组件最初并没有使用React.forwardRef来转发这个ref。

技术原理分析

在React中，ref是一种特殊的属性，它不能像普通props那样自动传递。当我们需要在函数组件上使用ref时，必须使用React.forwardRef来显式地转发ref。这是React的设计决策，目的是让开发者明确知道哪些组件需要处理ref逻辑。

在shadcn-ui的案例中，DropdownMenuTrigger组件使用了Radix UI的设计模式，它要求触发元素能够接收ref。当使用asChild属性时，这种要求变得更加严格，因为Radix需要完全控制DOM元素的引用。

解决方案实现

针对这个问题，我们可以通过修改SidebarMenuAction组件的实现来解决。以下是改进后的组件代码：

const SidebarMenuAction = React.forwardRef<
  HTMLButtonElement,
  React.ComponentProps<"button"> & {
    asChild?: boolean;
    showOnHover?: boolean;
  }
>(({ className, asChild = false, showOnHover = false, ...props }, ref) => {
  const Comp = asChild ? Slot : "button";

  return (
    <Comp
      ref={ref}
      data-slot="sidebar-menu-action"
      data-sidebar="menu-action"
      className={cn(
        "text-sidebar-foreground ring-sidebar-ring hover:bg-sidebar-accent hover:text-sidebar-accent-foreground peer-hover/menu-button:text-sidebar-accent-foreground absolute top-1.5 right-1 flex aspect-square w-5 items-center justify-center rounded-md p-0 outline-hidden transition-transform focus-visible:ring-2 [&>svg]:size-4 [&>svg]:shrink-0",
        "after:absolute after:-inset-2 md:after:hidden",
        "peer-data-[size=sm]/menu-button:top-1",
        "peer-data-[size=default]/menu-button:top-1.5",
        "peer-data-[size=lg]/menu-button:top-2.5",
        "group-data-[collapsible=icon]:hidden",
        showOnHover &&
          "peer-data-[active=true]/menu-button:text-sidebar-accent-foreground group-focus-within/menu-item:opacity-100 group-hover/menu-item:opacity-100 data-[state=open]:opacity-100 md:opacity-0",
        className
      )}
      {...props}
    />
  );
});