@homebound/beam 2.156.1 → 2.157.2

package/dist/components/ButtonMenu.d.ts

@@ -10,6 +10,7 @@ declare type MenuItemBase = {
     label: string;
     onClick: string | VoidFunction;
     disabled?: boolean;
+    destructive?: boolean;
 };
 export declare type IconMenuItemType = MenuItemBase & {
     icon: IconProps["icon"];

package/dist/components/SuperDrawer/SuperDrawer.js

@@ -33,6 +33,7 @@ const utils_1 = require("../../utils");
  * above the SuperDrawer.
  */
 function SuperDrawer() {
+    var _a, _b;
     const { drawerContentStack: contentStack, modalState, modalBodyDiv, modalFooterDiv, modalHeaderDiv, } = (0, BeamContext_1.useBeamContext)();
     const { closeDrawer } = (0, components_1.useSuperDrawer)();
     const drawerHeaderRef = (0, react_3.useRef)(null);
@@ -48,17 +49,16 @@ function SuperDrawer() {
         }
         // eslint-disable-next-line react-hooks/exhaustive-deps
     }, [modalBodyRef.current, modalFooterRef.current, modalState.current]);
-    if (contentStack.current.length === 0) {
-        return null;
-    }
     // Get the latest element on the stack
-    const currentContent = contentStack.current[contentStack.current.length - 1].opts;
-    const { content } = currentContent;
+    // We use undefined, nullish operators and empty object here to allow AnimatePresence
+    // to animate the drawers exit transition when our stack is empty
+    const currentContent = (_a = contentStack.current[contentStack.current.length - 1]) === null || _a === void 0 ? void 0 : _a.opts;
+    const { content } = currentContent !== null && currentContent !== void 0 ? currentContent : {};
     // Also get the first / non-detail element on the stack
-    const firstContent = contentStack.current[0].opts;
-    const { onPrevClick, onNextClick, titleRightContent, titleLeftContent, hideControls } = firstContent;
+    const firstContent = (_b = contentStack.current[0]) === null || _b === void 0 ? void 0 : _b.opts;
+    const { onPrevClick, onNextClick, titleRightContent, titleLeftContent, hideControls } = firstContent !== null && firstContent !== void 0 ? firstContent : {};
     const isDetail = currentContent !== firstContent;
-    const title = currentContent.title || firstContent.title;
+    const title = content === undefined ? '' : currentContent.title || firstContent.title;
     return (0, react_dom_1.createPortal)((0, jsx_runtime_1.jsx)(framer_motion_1.AnimatePresence, { children: content && ((0, jsx_runtime_1.jsxs)(jsx_runtime_1.Fragment, { children: [(0, jsx_runtime_1.jsx)(react_2.Global, { styles: { body: Css_1.Css.overflowHidden.$ } }, void 0), (0, react_1.createElement)(framer_motion_1.motion.div, { ...testId, 
                     // Key is required for framer-motion animations
                     key: "superDrawer", 

package/dist/components/SuperDrawer/SuperDrawerContent.js

@@ -15,10 +15,11 @@ const Css_1 = require("../../Css");
  * that knows how to handle the title, prev/next link and the onClose handler.
  */
 const SuperDrawerContent = ({ children, actions }) => {
+    var _a;
     const { closeDrawerDetail } = (0, useSuperDrawer_1.useSuperDrawer)();
     const { drawerContentStack: contentStack } = (0, BeamContext_1.useBeamContext)();
     // Determine if the current element is a new content element or an detail element
-    const { kind } = contentStack.current[contentStack.current.length - 1];
+    const { kind } = (_a = contentStack.current[contentStack.current.length - 1]) !== null && _a !== void 0 ? _a : {};
     function wrapWithMotionAndMaybeBack(children) {
         if (kind === "open") {
             return ((0, jsx_runtime_1.jsx)(framer_motion_1.motion.div, Object.assign({ css: Css_1.Css.p3.fg1.overflowAuto.$ }, { children: children }), "content"));

package/dist/components/Table/sortRows.js

@@ -23,7 +23,10 @@ function sortBatch(columns, batch, sortState, caseSensitive) {
     const primaryColumn = primaryKey && columns[primaryKey];
     // Make a shallow copy for sorting to avoid mutating the original list
     return [...batch].sort((a, b) => {
-        if ((a.pin || b.pin) && !(a.pin === b.pin)) {
+        if (a.pin || b.pin) {
+            // If both rows are pinned, we don't sort within them, because by pinning the page is taking
+            // explicit ownership over the order of the rows (and we also don't support "levels of pins",
+            // i.e. for change events putting "just added" rows `pin: last` and the "add new" row `pin: lastest`).
             const ap = a.pin === "first" ? -1 : a.pin === "last" ? 1 : 0;
             const bp = b.pin === "first" ? -1 : b.pin === "last" ? 1 : 0;
             return ap === bp ? 0 : ap < bp ? -1 : 1;

package/dist/components/internal/MenuItem.js

@@ -14,7 +14,7 @@ const defaultTestId_1 = require("../../utils/defaultTestId");
 function MenuItemImpl(props) {
     const { item, state, onClose } = props;
     const menuItem = item.value;
-    const { disabled: isDisabled, onClick, label } = menuItem;
+    const { disabled: isDisabled, onClick, label, destructive } = menuItem;
     const isFocused = state.selectionManager.focusedKey === item.key;
     const ref = (0, react_1.useRef)(null);
     const history = (0, react_router_1.useHistory)();
@@ -43,6 +43,7 @@ function MenuItemImpl(props) {
             ...(!isDisabled && isHovered ? Css_1.Css.bgGray100.$ : {}),
             ...(isFocused ? Css_1.Css.add("boxShadow", `inset 0 0 0 1px ${Css_1.Palette.LightBlue700}`).$ : {}),
             ...(isDisabled ? Css_1.Css.gray500.cursorNotAllowed.$ : {}),
+            ...(destructive ? Css_1.Css.red600.$ : {}),
         } }, tid[(0, defaultTestId_1.defaultTestId)(menuItem.label)], { children: maybeWrapInLink(onClick, isIconMenuItem(menuItem) ? ((0, jsx_runtime_1.jsx)(IconMenuItem, Object.assign({}, menuItem), void 0)) : isImageMenuItem(menuItem) ? ((0, jsx_runtime_1.jsx)(ImageMenuItem, Object.assign({}, menuItem), void 0)) : (label), isDisabled) }), void 0));
 }
 exports.MenuItemImpl = MenuItemImpl;

package/package.json

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