@homebound/beam 2.374.0 → 2.376.0

package/dist/components/Table/GridTableApi.js

@@ -8,6 +8,7 @@ const mobx_utils_1 = require("mobx-utils");
 const react_1 = require("react");
 const index_1 = require("../index");
 const TableState_1 = require("./utils/TableState");
+const utils_1 = require("../../utils");
 /**
  * Creates an `api` handle to drive a `GridTable`.
  *
@@ -152,7 +153,7 @@ class GridTableApiImpl {
                     if (!(0, index_1.isJSX)(content))
                         return content;
                     // Otherwise use the value/sortValue values
-                    return cell.value ? maybeApply(cell.value) : cell.sortValue ? maybeApply(cell.sortValue) : "-";
+                    return (0, utils_1.isDefined)(cell.value) ? maybeApply(cell.value) : cell.sortValue ? maybeApply(cell.sortValue) : "-"; // Do we need the "-" handling unclear if we use it ever
                 }
                 else {
                     // ReactNode

package/dist/inputs/internal/ComboBoxBase.js

@@ -1,14 +1,10 @@
 "use strict";
-var __importDefault = (this && this.__importDefault) || function (mod) {
-    return (mod && mod.__esModule) ? mod : { "default": mod };
-};
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.addNewOption = exports.unsetOption = void 0;
 exports.ComboBoxBase = ComboBoxBase;
 exports.initializeOptions = initializeOptions;
 exports.disabledOptionToKeyedTuple = disabledOptionToKeyedTuple;
 const jsx_runtime_1 = require("@emotion/react/jsx-runtime");
-const fast_deep_equal_1 = __importDefault(require("fast-deep-equal"));
 const react_1 = require("react");
 const react_aria_1 = require("react-aria");
 const react_stately_1 = require("react-stately");
@@ -68,18 +64,7 @@ function ComboBoxBase(props) {
         : [propOptions.current, propOptions.options, unsetLabel, onAddNew]);
     const values = (0, react_1.useMemo)(() => propValues !== null && propValues !== void 0 ? propValues : [], [propValues]);
     const inputStylePalette = (0, react_1.useMemo)(() => propsInputStylePalette, [propsInputStylePalette]);
-    const selectedOptionsRef = (0, react_1.useRef)(options.filter((o) => values.includes(getOptionValue(o))));
-    const selectedOptions = (0, react_1.useMemo)(() => {
-        // `selectedOptions` should only ever update if the `values` prop actually change.
-        // Assuming `values` is a state variable, then it should hold its identity until it _really_ changes.
-        // Though, it is possible that the `options` prop has changed, which is a dependency on this `useMemo`.
-        // That could trigger an unnecessary new reference for `selectedOptions`, and cause additional renders or unexpected state changes.
-        // We should avoid updating `selectedOptions` unless `values` has actually changed.
-        if (!(0, fast_deep_equal_1.default)([...values].sort(), selectedOptionsRef.current.map(getOptionValue).sort())) {
-            selectedOptionsRef.current = options.filter((o) => values.includes(getOptionValue(o)));
-        }
-        return selectedOptionsRef.current;
-    }, [options, values, getOptionValue]);
+    const selectedOptions = (0, react_1.useMemo)(() => options.filter((o) => values.includes(getOptionValue(o))), [options, values, getOptionValue]);
     const { contains } = (0, react_aria_1.useFilter)({ sensitivity: "base" });
     const isDisabled = !!disabled;
     const isReadOnly = !!readOnly;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@homebound/beam",
-  "version": "2.374.0",
+  "version": "2.376.0",
   "author": "Homebound",
   "license": "MIT",
   "main": "dist/index.js",