@homebound/beam 2.283.0 → 2.284.0

package/dist/components/ButtonGroup.d.ts

@@ -10,7 +10,7 @@ export type ButtonGroupButton = {
     icon?: IconProps["icon"];
     iconColor?: IconProps["color"];
     iconInc?: IconProps["inc"];
-    text?: string;
+    text?: ReactNode;
     onClick?: VoidFunction;
     /** Disables the button. Pass a ReactNode to disable the button and show a tooltip */
     disabled?: boolean | ReactNode;

package/dist/components/ButtonGroup.js

@@ -20,7 +20,6 @@ function ButtonGroup(props) {
 }
 exports.ButtonGroup = ButtonGroup;
 function GroupButton(props) {
-    var _a;
     const { icon, iconInc, iconColor, text, active, onClick: onPress, disabled, size, tooltip, ...otherProps } = props;
     const ariaProps = { onPress, isDisabled: !!disabled, ...otherProps };
     const ref = (0, react_1.useRef)(null);
@@ -38,7 +37,7 @@ function GroupButton(props) {
                     ...(active ? activeStyles : {}),
                     ...(isPressed ? pressedStyles : isHovered ? hoverStyles : {}),
                     ...(icon ? iconStyles[size] : {}),
-                }, ...tid[(0, defaultTestId_1.defaultTestId)((_a = text !== null && text !== void 0 ? text : icon) !== null && _a !== void 0 ? _a : "button")], children: [icon && ((0, jsx_runtime_1.jsx)(Icon_1.Icon, { xss: Css_1.Css.if(!!text).mrPx(4).$, icon: icon, color: disabled ? undefined : iconColor, inc: iconInc })), text] })),
+                }, ...tid[(0, defaultTestId_1.defaultTestId)((typeof text === "string" && text) || icon || "button")], children: [icon && ((0, jsx_runtime_1.jsx)(Icon_1.Icon, { xss: Css_1.Css.if(!!text).mrPx(4).$, icon: icon, color: disabled ? undefined : iconColor, inc: iconInc })), text] })),
         }) }));
 }
 const pressedStyles = Css_1.Css.bgGray200.$;

package/dist/inputs/MultiSelectField.js

@@ -6,16 +6,7 @@ const ComboBoxBase_1 = require("./internal/ComboBoxBase");
 function MultiSelectField(props) {
     const { getOptionValue = (opt) => opt.id, // if unset, assume O implements HasId
     getOptionLabel = (opt) => opt.name, // if unset, assume O implements HasName
-    options, onSelect, values, ...otherProps } = props;
-    return ((0, jsx_runtime_1.jsx)(ComboBoxBase_1.ComboBoxBase, { multiselect: true, ...otherProps, options: options, getOptionLabel: getOptionLabel, getOptionValue: getOptionValue, values: values, onSelect: (values) => {
-            const [selectedValues, selectedOptions] = options
-                .filter((o) => values.includes(getOptionValue(o)))
-                .reduce((acc, o) => {
-                acc[0].push(getOptionValue(o));
-                acc[1].push(o);
-                return acc;
-            }, [[], []]);
-            onSelect(selectedValues, selectedOptions);
-        } }));
+    ...otherProps } = props;
+    return (0, jsx_runtime_1.jsx)(ComboBoxBase_1.ComboBoxBase, { multiselect: true, getOptionLabel: getOptionLabel, getOptionValue: getOptionValue, ...otherProps });
 }
 exports.MultiSelectField = MultiSelectField;

package/dist/inputs/SelectField.d.ts

@@ -5,6 +5,11 @@ import { HasIdAndName, Optional } from "../types";
 export interface SelectFieldProps<O, V extends Value> extends Omit<ComboBoxBaseProps<O, V>, "values" | "onSelect" | "multiselect"> {
     /** The current value; it can be `undefined`, even if `V` cannot be. */
     value: V | undefined;
+    /**
+     * Called when a value is selected, or `undefined` if `unsetLabel` is being used.
+     *
+     * Ideally callers that didn't pass `unsetLabel` would not have to handle the ` | undefined` here.
+     */
     onSelect: (value: V | undefined, opt: O | undefined) => void;
 }
 /**

package/dist/inputs/SelectField.js

@@ -8,6 +8,7 @@ function SelectField(props) {
     getOptionLabel = (opt) => opt.name, // if unset, assume O implements HasName
     options, onSelect, value, ...otherProps } = props;
     return ((0, jsx_runtime_1.jsx)(ComboBoxBase_1.ComboBoxBase, { ...otherProps, options: options, getOptionLabel: getOptionLabel, getOptionValue: getOptionValue, values: [value], onSelect: (values, options) => {
+            // If the user used `unsetLabel`, then values will be `[undefined]` and options `[unsetOption]`
             if (values.length > 0 && options.length > 0) {
                 const option = options[0];
                 onSelect(values[0], option === ComboBoxBase_1.unsetOption ? undefined : option);

package/dist/inputs/internal/ComboBoxBase.d.ts

@@ -29,13 +29,25 @@ export interface ComboBoxBaseProps<O, V extends Value> extends BeamFocusableProp
     onBlur?: () => void;
     onFocus?: () => void;
     sizeToContent?: boolean;
-    /** The text to show when nothing is selected, i.e. could be "All" for filters. */
+    /**
+     * The text to show when nothing is selected, i.e. could be "All" for filters.
+     *
+     * Unlike `unsetLabel`, this does not add an explicit option for the user to select.
+     */
     nothingSelectedText?: string;
     /** When set the SelectField is expected to be put on a darker background */
     contrast?: boolean;
     /** Placeholder content */
     placeholder?: string;
-    /** Only used for Single Select Fields. If set, prepends an option with a `undefined` value at the top of the list */
+    /**
+     * If set, prepends an option with an `undefined` value at the top of the list to allow
+     * unsetting the field.
+     *
+     * Unlike `nothingSelectedText`, which provides the label for empty value state, but doesn't
+     * add an option for the user to explicitly select the empty state.
+     *
+     * Only available for Single Select Fields.
+     */
     unsetLabel?: string;
     hideErrorMessage?: boolean;
 }

package/package.json

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