@homebound/beam 4.40.0 → 4.40.1

package/dist/index.js

@@ -6283,7 +6283,7 @@ var ColumnState = class {
   constructor(states, storage, column2) {
     this.states = states;
     this.column = column2;
-    this.visible = storage.wasVisible(column2.id) ?? (column2.canHide ? !(column2.initHidden ?? true) : true);
+    this.visible = !column2.canHide || (storage.wasVisible(column2.id) ?? !(column2.initHidden ?? true));
     if (this.visible && (storage.wasExpanded(column2.id) ?? column2.initExpanded)) {
       this.expanded = true;
       this.doExpand();
@@ -6294,10 +6294,17 @@ var ColumnState = class {
   children = void 0;
   visible = true;
   expanded = false;
+  /**
+   * Whether this column is shown. We check `canHide` here too b/c when `props.columns` changes,
+   * `addColumn` can replace a hidden column's definition with one that can't be hidden.
+   */
+  get isVisible() {
+    return this.visible || !this.column.canHide;
+  }
   setVisible(visible) {
-    const wasVisible = this.visible;
-    this.visible = visible;
-    if (!wasVisible && visible && this.column.initExpanded && this.children === void 0) {
+    const wasVisible = this.isVisible;
+    this.visible = visible || !this.column.canHide;
+    if (!wasVisible && this.visible && this.column.initExpanded && this.children === void 0) {
       this.expanded = true;
       this.doExpand();
     }
@@ -6321,9 +6328,13 @@ var ColumnState = class {
       this.children = expandColumns.map((ec) => this.states.addColumn(ec));
     }
   }
+  /** Returns this column and all of its loaded children, whether visible and expanded or not. */
+  get selfAndChildren() {
+    return [this, ...(this.children ?? []).flatMap((c) => c.selfAndChildren)];
+  }
   /** Returns this column, if visible, and its children, if expanded. */
   get maybeSelfAndChildren() {
-    if (!this.visible) {
+    if (!this.isVisible) {
       return [];
     } else if (this.expanded && this.children) {
       return [this, ...this.children.flatMap((c) => c.maybeSelfAndChildren)];
@@ -7573,9 +7584,13 @@ var ColumnStates = class {
   allVisibleColumns(showIn) {
     return this.columns.flatMap((cs) => cs.maybeSelfAndChildren).filter((cs) => !cs.column.showIn || cs.column.showIn === showIn).filter((cs) => showIn === "csv" ? isContentColumn(cs.column) : true);
   }
+  /**
+   * Shows the current columns in `ids` and hides the others. Columns that can't be hidden, like layout gutters, stay shown.
+   * Columns not currently in the table keep their visibility, so a column that's temporarily removed comes back as it was.
+   */
   setVisibleColumns(ids) {
-    for (const cs of this.map.values()) {
-      cs.setVisible(cs.column.isLayoutGutter ? true : ids.includes(cs.column.id));
+    for (const cs of this.columns.flatMap((c) => c.selfAndChildren)) {
+      cs.setVisible(ids.includes(cs.column.id));
     }
   }
   loadExpanded(storageKey) {