From 302f09b5b9d2e1127f94eb4e0dcea8335bf22d9d Mon Sep 17 00:00:00 2001 From: Atul Tameshwari Date: Thu, 27 Aug 2026 14:54:18 +0530 Subject: [PATCH] fix: remove redundant Fragment wrapper around comboButton in dropdowns (#9694) Combobox.Button as={Fragment} requires its child to be a single real element. Wrapping the button ternary in <>... made the child a Fragment instance itself, which @headlessui/react v2 rejects with "Passing props on Fragment!" at runtime. Dropping the wrapper fixes priority, estimate, intake-state, member, module, cycle, state, and project dropdowns. --- apps/web/core/components/dropdowns/cycle/index.tsx | 8 ++------ apps/web/core/components/dropdowns/estimate.tsx | 8 ++------ apps/web/core/components/dropdowns/intake-state/base.tsx | 8 ++------ apps/web/core/components/dropdowns/member/base.tsx | 8 ++------ apps/web/core/components/dropdowns/module/base.tsx | 8 ++------ apps/web/core/components/dropdowns/priority.tsx | 8 ++------ apps/web/core/components/dropdowns/project/base.tsx | 8 ++------ apps/web/core/components/dropdowns/state/base.tsx | 8 ++------ 8 files changed, 16 insertions(+), 48 deletions(-) diff --git a/apps/web/core/components/dropdowns/cycle/index.tsx b/apps/web/core/components/dropdowns/cycle/index.tsx index a2a9919b1b..49921d97eb 100644 --- a/apps/web/core/components/dropdowns/cycle/index.tsx +++ b/apps/web/core/components/dropdowns/cycle/index.tsx @@ -83,9 +83,7 @@ export const CycleDropdown = observer(function CycleDropdown(props: Props) { handleClose(); }; - const comboButton = ( - <> - {button ? ( + const comboButton = button ? ( - )} - - ); + ); return ( - {button ? ( + const comboButton = button ? ( - )} - - ); + ); return ( - {button ? ( + const comboButton = button ? ( - )} - - ); + ); return ( // oxlint-disable-next-line jsx_a11y/no-static-element-interactions diff --git a/apps/web/core/components/dropdowns/member/base.tsx b/apps/web/core/components/dropdowns/member/base.tsx index 577e60a781..cd0e043cf4 100644 --- a/apps/web/core/components/dropdowns/member/base.tsx +++ b/apps/web/core/components/dropdowns/member/base.tsx @@ -108,9 +108,7 @@ export const MemberDropdownBase = observer(function MemberDropdownBase(props: TM } }; - const comboButton = ( - <> - {button ? ( + const comboButton = button ? ( - )} - - ); + ); return ( - {button ? ( + const comboButton = button ? ( - )} - - ); + ); return ( - {button ? ( + const comboButton = button ? ( - )} - - ); + ); return ( - {button ? ( + const comboButton = button ? ( - )} - - ); + ); return ( - {button ? ( + const comboButton = button ? ( - )} - - ); + ); return ( // oxlint-disable-next-line jsx_a11y/no-static-element-interactions