fix: itemToKey by value (#3158)

Author: rivka-ungar

packages/core/src/components/next/Dropdown/hooks/useDropdownCombobox.ts

@@ -54,6 +54,7 @@ function useDropdownCombobox<T extends BaseListItemData<Record<string, unknown>>
   } = useCombobox<T>({
     items: flatOptions,
     itemToString: item => item?.label ?? "",
+    itemToKey: item => (item?.value !== undefined ? String(item.value) : ""),
     isItemDisabled: item => Boolean(item.disabled),
     initialInputValue: inputValueProp || "",
     selectedItem: selectedItem,

packages/core/src/components/next/Dropdown/hooks/useDropdownMultiCombobox.ts

@@ -59,6 +59,7 @@ function useDropdownMultiCombobox<T extends BaseListItemData<Record<string, unkn
   } = useCombobox<T>({
     items: flatOptions,
     itemToString: item => item?.label ?? "",
+    itemToKey: item => (item?.value !== undefined ? String(item.value) : ""),
     isItemDisabled: item => Boolean(item.disabled),
     isOpen: isMenuOpen,
     initialIsOpen: autoFocus,

packages/core/src/components/next/Dropdown/hooks/useDropdownMultiSelect.ts

@@ -51,6 +51,7 @@ function useDropdownMultiSelect<T extends BaseListItemData<Record<string, unknow
   } = useSelect<T>({
     items: flatOptions,
     itemToString: item => item?.label ?? "",
+    itemToKey: item => (item?.value !== undefined ? String(item.value) : ""),
     isItemDisabled: item => Boolean(item.disabled),
     selectedItem: null,
     isOpen: isMenuOpen,

packages/core/src/components/next/Dropdown/hooks/useDropdownSelect.ts

@@ -49,6 +49,7 @@ function useDropdownSelect<T extends BaseListItemData<Record<string, unknown>>>(
   } = useSelect<T>({
     items: flatOptions,
     itemToString: item => item?.label ?? "",
+    itemToKey: item => (item?.value !== undefined ? String(item.value) : ""),
     isItemDisabled: item => Boolean(item.disabled),
     isOpen: isMenuOpen,
     initialIsOpen: autoFocus,