@eightshift/ui-components 1.4.4 → 1.4.6

package/dist/components/select/async-multi-select.js

@@ -92,8 +92,11 @@ const AsyncMultiSelect = (props) => {
   const idBase = useId();
   const value = rawValue.map((item, index) => ({
     ...item,
-    id: `${idBase}-${index}`
+    id: (item == null ? void 0 : item.value) ?? `${idBase}-${index}`
   }));
+  const modifiedOnChange = (v) => {
+    onChange(v == null ? void 0 : v.map((item) => ({ ...item, id: void 0 })));
+  };
   if (hidden) {
     return null;
   }
@@ -114,7 +117,7 @@ const AsyncMultiSelect = (props) => {
         DndContext,
         {
           modifiers: [restrictToParentElement],
-          onDragEnd: getDragEndHandler(onChange, value),
+          onDragEnd: getDragEndHandler(modifiedOnChange, value),
           children: /* @__PURE__ */ jsx(SortableContext, { items: value.map(({ id }) => id), children: /* @__PURE__ */ jsx(
             AsyncSelect$1,
             {
@@ -123,7 +126,7 @@ const AsyncMultiSelect = (props) => {
               loadOptions: customLoadOptions,
               defaultOptions: preloadOptions,
               value,
-              onChange,
+              onChange: modifiedOnChange,
               closeMenuOnSelect: !keepMenuOpenAfterSelect,
               isClearable: clearable,
               isSearchable: !noSearch,

package/dist/components/select/async-single-select.js

@@ -105,7 +105,10 @@ const AsyncSelect = (props) => {
           loadOptions: customLoadOptions,
           defaultOptions: preloadOptions,
           value,
-          onChange,
+          onChange: (value2) => {
+            delete value2.id;
+            onChange(value2);
+          },
           closeMenuOnSelect: !keepMenuOpenAfterSelect,
           isClearable: clearable,
           isSearchable: !noSearch,

package/dist/components/select/multi-select.js

@@ -4,7 +4,7 @@ import { S as StateManagedSelect$1 } from "../../react-select.esm-DkSeQzOP.js";
 import { D as DndContext, g as getDragEndHandler, S as SortableContext, a as getMultiValue, b as getMultiValueRemove } from "../../multi-select-components-DTvEidE3.js";
 import { r as restrictToParentElement } from "../../modifiers.esm-BuJQPI1X.js";
 import { CustomSelectDefaultMultiValueRemove, CustomSelectDefaultDropdownIndicator, CustomSelectDefaultClearIndicator } from "./custom-select-default-components.js";
-import { getValue, customOnChange } from "./shared.js";
+import { getValue } from "./shared.js";
 import { BaseControl } from "../base-control/base-control.js";
 import { eightshiftSelectClasses } from "./styles.js";
 import { c as components } from "../../index-a301f526.esm-B9tWL9yi.js";
@@ -87,8 +87,15 @@ const MultiSelect = (props) => {
   const idBase = useId();
   const value = getValue(simpleValue, rawValue, options).map((item, index) => ({
     ...item,
-    id: `${idBase}-${index}`
+    id: (simpleValue ? item : item == null ? void 0 : item.value) ?? `${idBase}-${index}`
   }));
+  const modifiedOnChange = (v) => {
+    if (simpleValue) {
+      onChange(v == null ? void 0 : v.map(({ value: value2 }) => value2));
+      return;
+    }
+    onChange(v == null ? void 0 : v.map((item) => ({ ...item, id: void 0 })));
+  };
   if (hidden) {
     return null;
   }
@@ -105,7 +112,7 @@ const MultiSelect = (props) => {
         DndContext,
         {
           modifiers: [restrictToParentElement],
-          onDragEnd: getDragEndHandler(onChange, value),
+          onDragEnd: getDragEndHandler(modifiedOnChange, value),
           children: /* @__PURE__ */ jsx(SortableContext, { items: value.map(({ id }) => id), children: /* @__PURE__ */ jsx(
             StateManagedSelect$1,
             {
@@ -113,7 +120,7 @@ const MultiSelect = (props) => {
               unstyled: true,
               options: options.map((item) => ({ id: item.value, ...item })),
               value,
-              onChange: (v) => customOnChange(simpleValue, v, onChange),
+              onChange: modifiedOnChange,
               closeMenuOnSelect: !keepMenuOpenAfterSelect,
               isClearable: clearable,
               isSearchable: !noSearch,

package/dist/components/select/shared.js

@@ -10,66 +10,17 @@
  * @preserve
  */
 const getValue = (simpleValue, value, options) => {
-  if (!simpleValue) {
-    return value;
-  }
   if (Array.isArray(value)) {
-    return value.map((value2) => options == null ? void 0 : options.find(({ value: itemValue }) => itemValue === value2));
-  }
-  return options == null ? void 0 : options.find(({ value: itemValue }) => itemValue === value);
-};
-/**
- * Handles the `onChange` callback.
- *
- * @param {boolean} simpleValue - Whether `simpleValue` is set.
- * @param {string|{label: string, value: string, metadata: Object<string, any>[]}} newValue - The new value to be set.
- * @param {Function} onChange - The `onChange` callback passed to the component.
- * @returns {void}
- *
- * @preserve
- */
-const customOnChange = (simpleValue, newValue, onChange) => {
-  if (typeof newValue === "undefined" || newValue === null || newValue === "") {
-    onChange(void 0);
-    return;
-  }
-  delete newValue.id;
-  if (!simpleValue) {
-    onChange(newValue);
-    return;
-  }
-  if (Array.isArray(newValue)) {
-    onChange(newValue.map((item) => item == null ? void 0 : item.value));
-    return;
+    if (simpleValue) {
+      return value.map((value2) => options == null ? void 0 : options.find(({ value: itemValue }) => itemValue === value2));
+    }
+    return value;
   }
-  onChange(newValue == null ? void 0 : newValue.value);
-};
-/**
- * Handles the `onChange` callback.
- *
- * @param {object[]} items - Current items.
- * @param {Function} onChange - The `onChange` callback passed to the component.
- * @returns {void}
- *
- * @preserve
- */
-const fixIds = (items, onChange) => {
-  const allIds = (items == null ? void 0 : items.map(({ id }) => id)) ?? [];
-  const hasDuplicates = (input) => {
-    var _a;
-    return ((_a = new Set(input)) == null ? void 0 : _a.size) !== (input == null ? void 0 : input.length);
-  };
-  const hasMissingIds = items == null ? void 0 : items.some(({ id }) => typeof id === "undefined" || id === null || id === "");
-  if (hasDuplicates(allIds) && (items == null ? void 0 : items.length) > 0 || hasMissingIds) {
-    const newItems = [...items].map((item, index) => ({
-      ...item,
-      id: index + 1
-    }));
-    onChange(newItems);
+  if (simpleValue) {
+    return options == null ? void 0 : options.find(({ value: itemValue }) => itemValue === value);
   }
+  return value;
 };
 export {
-  customOnChange,
-  fixIds,
   getValue
 };

package/dist/components/select/single-select.js

@@ -1,7 +1,7 @@
 import { jsx } from "react/jsx-runtime";
 import { S as StateManagedSelect$1 } from "../../react-select.esm-DkSeQzOP.js";
 import { CustomSelectDefaultDropdownIndicator, CustomSelectDefaultClearIndicator } from "./custom-select-default-components.js";
-import { getValue, customOnChange } from "./shared.js";
+import { getValue } from "./shared.js";
 import { BaseControl } from "../base-control/base-control.js";
 import { eightshiftSelectClasses } from "./styles.js";
 import { c as components } from "../../index-a301f526.esm-B9tWL9yi.js";
@@ -95,7 +95,14 @@ const Select = (props) => {
           unstyled: true,
           options,
           value: getValue(simpleValue, value, options),
-          onChange: (v) => customOnChange(simpleValue, v, onChange),
+          onChange: (v) => {
+            if (simpleValue) {
+              onChange(v == null ? void 0 : v.value);
+              return;
+            }
+            delete v.id;
+            onChange(v);
+          },
           closeMenuOnSelect: !keepMenuOpenAfterSelect,
           isClearable: clearable,
           isSearchable: !noSearch,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@eightshift/ui-components",
-  "version": "1.4.4",
+  "version": "1.4.6",
   "type": "module",
   "main": "./dist/index.js",
   "module": "./dist/index.js",
@@ -53,29 +53,29 @@
     "@dnd-kit/utilities": "^3.2.2",
     "@eslint/compat": "^1.1.1",
     "@react-stately/collections": "^3.10.9",
-    "@stylistic/eslint-plugin-js": "^2.3.0",
+    "@stylistic/eslint-plugin-js": "^2.6.1",
     "@types/react": "^18.3.3",
     "@types/react-dom": "^18.3.0",
     "@vitejs/plugin-react-swc": "^3.7.0",
     "@wordpress/i18n": "^5.4.0",
-    "autoprefixer": "^10.4.19",
+    "autoprefixer": "^10.4.20",
     "clsx": "^2.1.1",
     "css-gradient-parser": "^0.0.16",
-    "eslint": "^9.7.0",
+    "eslint": "^9.8.0",
     "eslint-config-prettier": "^9.1.0",
-    "eslint-plugin-jsdoc": "^48.8.3",
+    "eslint-plugin-jsdoc": "^50.0.0",
     "eslint-plugin-prettier": "^5.2.1",
-    "framer-motion": "^11.3.17",
+    "framer-motion": "^11.3.22",
     "glob": "^11.0.0",
-    "globals": "^15.8.0",
+    "globals": "^15.9.0",
     "just-camel-case": "^6.2.0",
     "just-debounce-it": "^3.2.0",
     "just-has": "^2.3.0",
     "just-is-empty": "^3.4.1",
     "just-kebab-case": "^4.2.0",
     "just-throttle": "^4.2.0",
-    "lightningcss": "^1.25.1",
-    "postcss": "^8.4.40",
+    "lightningcss": "^1.26.0",
+    "postcss": "^8.4.41",
     "prettier": "^3.3.3",
     "prettier-plugin-tailwindcss": "^0.6.5",
     "react-aria": "^3.34.1",

package/dist/hooks/use-cell-edit-mode.js

@@ -1,35 +0,0 @@
-import { useRef, useEffect } from "react";
-function useCellEditMode() {
-  const isFocusedRef = useRef(false);
-  const isEditModeRef = useRef(false);
-  useEffect(() => {
-    const handler = (e) => {
-      if (isFocusedRef.current) {
-        if (e.code === "Enter") {
-          isEditModeRef.current = true;
-        } else if (e.code === "Escape" || e.code === "Tab") {
-          isEditModeRef.current = false;
-        }
-        if (isEditModeRef.current) {
-          e.stopPropagation();
-        }
-      }
-    };
-    window.addEventListener("keydown", handler, true);
-    return () => {
-      window.removeEventListener("keydown", handler, true);
-    };
-  }, []);
-  const setFocus = (should) => isFocusedRef.current = should;
-  const preventProps = {
-    onFocus: () => setFocus(true),
-    onBlur: () => setFocus(false),
-    onClick: () => {
-      isEditModeRef.current = true;
-    }
-  };
-  return preventProps;
-}
-export {
-  useCellEditMode
-};