From ed6966c4e6e1259d12ee503727492d5928da9f3c Mon Sep 17 00:00:00 2001 From: Florent Mathieu Date: Wed, 23 Oct 2024 13:28:12 -1000 Subject: [PATCH] refactor(dropdowns): use transient props where appropriate --- .../src/elements/combobox/Combobox.tsx | 52 ++++++++------- .../src/elements/combobox/Listbox.tsx | 12 ++-- .../src/elements/combobox/OptGroup.tsx | 6 +- .../src/elements/combobox/Option.tsx | 4 +- .../src/elements/combobox/OptionMeta.tsx | 2 +- packages/dropdowns/src/elements/menu/Item.tsx | 4 +- .../dropdowns/src/elements/menu/ItemGroup.tsx | 6 +- .../dropdowns/src/elements/menu/ItemMeta.tsx | 2 +- .../dropdowns/src/elements/menu/MenuList.tsx | 14 ++-- .../src/views/combobox/StyledCombobox.ts | 6 +- .../views/combobox/StyledFloatingListbox.ts | 12 ++-- .../src/views/combobox/StyledInput.ts | 14 ++-- .../src/views/combobox/StyledListbox.ts | 16 ++--- .../src/views/combobox/StyledOptGroup.ts | 2 +- .../src/views/combobox/StyledOption.ts | 10 +-- .../src/views/combobox/StyledOptionMeta.ts | 6 +- .../src/views/combobox/StyledTagsButton.ts | 2 +- .../src/views/combobox/StyledTrigger.ts | 64 +++++++++---------- .../src/views/combobox/StyledValue.ts | 22 +++---- .../dropdowns/src/views/menu/StyledMenu.ts | 6 +- 20 files changed, 133 insertions(+), 129 deletions(-) diff --git a/packages/dropdowns/src/elements/combobox/Combobox.tsx b/packages/dropdowns/src/elements/combobox/Combobox.tsx index 14f0dc2cc1f..83963e02d57 100644 --- a/packages/dropdowns/src/elements/combobox/Combobox.tsx +++ b/packages/dropdowns/src/elements/combobox/Combobox.tsx @@ -177,15 +177,6 @@ export const Combobox = forwardRef( 'Options' ); const triggerProps = { - isAutocomplete, - isBare, - isCompact, - isEditable, - isLabelHovered, - isMultiselectable, - maxHeight, - focusInset, - validation, ...(getTriggerProps({ onFocus: () => { if (!isDisabled) { @@ -214,10 +205,6 @@ export const Combobox = forwardRef( const inputProps = { 'aria-invalid': validation === 'error' || validation === 'warning', hidden: isInputHidden, - isBare, - isCompact, - isEditable, - isMultiselectable, placeholder, ...(getInputProps({ ...(_inputProps as IUseComboboxReturnValue['getInputProps']) @@ -266,12 +253,23 @@ export const Combobox = forwardRef( return ( - + {!!startIcon && ( @@ -291,7 +289,7 @@ export const Combobox = forwardRef(