Summary
Found two minor bugs in apps/web/src/components/ui/components/Dropdown.tsx:
- Invalid CSS Class Syntax:
CHECKBOX_ITEM_BOX_CLASS contained group-hover:not-hover:border-edge-muted which is invalid Tailwind v4 syntax.
- Duplicate Prop Spreading:
DropdownItem passed closeOnSelect={props.closeOnSelect} explicitly without extracting closeOnSelect in splitProps, causing duplicate prop spreading in {...rest}.
Expected behavior
- Checkbox items inside dropdowns should use valid Tailwind hover variants (
group-hover:border-edge-muted).
- Props in
DropdownItem should be cleanly destructured in splitProps to avoid duplicate prop spreading.
Proposed Fix
- Updated
CHECKBOX_ITEM_BOX_CLASS to valid Tailwind v4 class syntax.
- Added
closeOnSelect to splitProps in DropdownItem.
Summary
Found two minor bugs in
apps/web/src/components/ui/components/Dropdown.tsx:CHECKBOX_ITEM_BOX_CLASScontainedgroup-hover:not-hover:border-edge-mutedwhich is invalid Tailwind v4 syntax.DropdownItempassedcloseOnSelect={props.closeOnSelect}explicitly without extractingcloseOnSelectinsplitProps, causing duplicate prop spreading in{...rest}.Expected behavior
group-hover:border-edge-muted).DropdownItemshould be cleanly destructured insplitPropsto avoid duplicate prop spreading.Proposed Fix
CHECKBOX_ITEM_BOX_CLASSto valid Tailwind v4 class syntax.closeOnSelecttosplitPropsinDropdownItem.