@homebound/beam 2.132.0 → 2.132.3

package/dist/components/Table/GridTable.js

@@ -274,6 +274,11 @@ const renders = {
 /** Renders table using divs with flexbox rows, which is the default render */
 function renderDiv(style, id, columns, headerRows, totalsRows, filteredRows, firstRowMessage, _stickyHeader, firstLastColumnWidth, xss, _virtuosoRef, tableRef) {
     return ((0, jsx_runtime_1.jsxs)("div", Object.assign({ ref: tableRef, css: {
+            // Use `fit-content` to ensure the width of the table takes up the full width of its content.
+            // Otherwise, the table's width would be that of its container, which may not be as wide as the table itself.
+            // In cases where we have sticky columns on a very wide table, then the container which the columns "stick" to (which is the table),
+            // needs to be as wide as the table's content, or else we lose the "stickiness" once scrolling past width of the table's container.
+            ...Css_1.Css.mw("fit-content").$,
             /*
               Using (n + 3) here to target all rows that are after the first non-header row. Since n starts at 0, we can use
               the + operator as an offset.

package/dist/components/Table/RowState.js

@@ -126,19 +126,19 @@ class RowState {
     }
     // Should be called in an Observer/useComputed to trigger re-renders
     isCollapsed(id) {
-        return this.collapsedRows.has(id) || this.collapsedRows.has("header");
+        return this.collapsedRows.has(id);
     }
     toggleCollapsed(id) {
         const collapsedIds = [...this.collapsedRows.values()];
         // We have different behavior when going from expand/collapse all.
         if (id === "header") {
-            const isAllCollapsed = collapsedIds[0] === "header";
+            const isAllCollapsed = collapsedIds.includes("header");
             if (isAllCollapsed) {
                 // Expand all means keep `collapsedIds` empty
                 collapsedIds.splice(0, collapsedIds.length);
             }
             else {
-                // Otherwise push `header` on the list as a hint that we're in the collapsed-all state
+                // Otherwise push `header` to the list as a hint that we're in the collapsed-all state
                 collapsedIds.push("header");
                 // Find all non-leaf rows so that toggling "all collapsed" -> "all not collapsed" opens
                 // the parent rows of any level.
@@ -164,6 +164,16 @@ class RowState {
             else {
                 collapsedIds.splice(i, 1);
             }
+            // If all rows have been expanded individually, but the 'header' was collapsed, then remove the header from the collapsedIds so it reverts to the expanded state
+            if (collapsedIds.length === 1 && collapsedIds[0] === "header") {
+                collapsedIds.splice(0, 1);
+            }
+            else {
+                // If every top level child has been collapsed, then push "header" into the array to be considered collapsed as well.
+                if (this.rows.every((maybeParent) => (maybeParent.children ? collapsedIds.includes(maybeParent.id) : true))) {
+                    collapsedIds.push("header");
+                }
+            }
         }
         this.collapsedRows.replace(collapsedIds);
         if (this.persistCollapse) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@homebound/beam",
-  "version": "2.132.0",
+  "version": "2.132.3",
   "author": "Homebound",
   "license": "MIT",
   "main": "dist/index.js",
@@ -116,6 +116,7 @@
     "identity-obj-proxy": "^3.0.0",
     "jest": "^26.6.3",
     "jest-chain": "^1.1.5",
+    "jest-watch-typeahead": "^0.6.5",
     "mobx": "^6.3.2",
     "mobx-react": "^7.2.0",
     "prettier": "^2.2.1",