flexdesk 0.4.6 → 0.5.0

package/dist/wm.js

@@ -2,10 +2,11 @@ import {
   HelpModal,
   openForm,
   showContextMenu
-} from "./chunk-ELXVW542.js";
+} from "./chunk-24MS7W5Y.js";
 import {
-  ManagedWindow
-} from "./chunk-LH5TSOZW.js";
+  ManagedWindow,
+  isRowActivation
+} from "./chunk-6JSOVNID.js";
 import "./chunk-FL5KFNQH.js";
 import {
   NotebookTabBar
@@ -1708,7 +1709,7 @@ function attachLandingTableBehavior(host, getRow, entityActions, options = {}) {
       dispatch(actionBtn.dataset.twmAction, tr2 ? getRow(tr2.__rowIndex) : null);
       return;
     }
-    if (ev.target.closest("button, a, input, select, textarea")) return;
+    if (!isRowActivation(ev, "click", { scope: host })) return;
     const tr = bodyRowFor(ev);
     if (tr) dispatch("open", getRow(tr.__rowIndex));
   };
@@ -2450,6 +2451,9 @@ function _segments(kind, props, taxonomy, rootCrumb, rootLabel, navigate, trail)
   }
   return segs;
 }
+function renderBreadcrumb(container, segments) {
+  _renderInto(container, segments);
+}
 function _renderInto(container, segments) {
   container.innerHTML = "";
   if (!segments?.length) return;
@@ -3041,6 +3045,16 @@ var TileRenderer = class {
     const leaf = this.tree?.get?.(leafId);
     return leaf && leaf.kind === "leaf" ? this._leafKindKey(leaf) : null;
   }
+  /**
+   * C37. WHAT A LEAF'S CONTENT FACTORY RETURNED for the tab on screen, or
+   * `null` — so the window manager can ask the content a question about
+   * itself (`wm.navigateBack`: *"have you a level of your own to go up?"*)
+   * without reaching into the cache. Read-only; a leaf that has not been
+   * rendered yet has none.
+   */
+  contentOf(leafId) {
+    return this._leafCache.get(leafId)?.content ?? null;
+  }
   /**
    * C34. ACCEPT A PROPS WRITE THE CONTENT MADE ABOUT ITSELF, WITHOUT
    * REBUILDING THE TILE THAT MADE IT.
@@ -4159,10 +4173,21 @@ var WindowManager = class _WindowManager {
    *  Adding a new "Open from X" click that should be a back-able
    *  navigation just needs to use one of the in-tile routing
    *  methods (openFromContext / openInLeaf / navigateActiveTab) —
-   *  they all record history by default. */
+   *  they all record history by default.
+   *
+   *  (0) C37. BEFORE ALL OF THAT, THE CONTENT'S OWN LEVELS. A page can have
+   *  levels that are not tabs — a settings page whose section opened one
+   *  member in place of its list, an organization page showing one of its
+   *  projects — and Backspace must climb those before it walks the tile. The
+   *  content factory says so by returning `navigateBack()` from its mount: it
+   *  is asked first, in the scope Backspace acts on (`_backScope`, so a
+   *  floating window's content and the focused tile's alike), and `true`
+   *  means it climbed. `false`, or no such function, and the walk below runs
+   *  exactly as it always has. */
   navigateBack() {
     const scope = this._backScope();
     if (!scope) return;
+    if (this._contentNavigateBack(scope)) return;
     if (scope.leafId && this._popTabHistory(scope.leafId)) return;
     if (this._returnToOpenList(scope)) return;
     const active = this._scopeActiveTab(scope);
@@ -4199,21 +4224,36 @@ var WindowManager = class _WindowManager {
     this._navigateInScope(scope, kind, props);
   }
   /** True when `navigateBack` would do something user-visible — i.e.
-   *  the active tab has per-tab history, OR the current kind has a
-   *  taxonomy parent. Used by the breadcrumb to grey out / hide the
-   *  Back button when there's nowhere to go. */
+   *  the content has a level of its own above the one it shows (C37), the
+   *  active tab has per-tab history, OR the current kind has a taxonomy
+   *  parent. Used by the breadcrumb to grey out / hide the Back button when
+   *  there's nowhere to go.
+   *
+   *  IT ANSWERS FOR THE SCOPE BACKSPACE ACTS ON — `_backScope`'s: a floating
+   *  window last pressed in, else the focused content tile, else the primary
+   *  one. It used to pick the tile alone, so after a press inside a window it
+   *  reported on the tile BEHIND it while Backspace climbed the window, and
+   *  C37's question went to the wrong content. For a tile the answer is
+   *  exactly what it always was; only a window scope answers differently. */
   canNavigateBack() {
-    const tree = this._tree();
-    const focusedId = tree.focusedLeafId;
-    const primaryId = tree.primaryLeafId();
-    const focusedLeaf = focusedId ? tree.get(focusedId) : null;
-    const focusedKind = focusedLeaf?.content?.kind || "";
-    const id = focusedId && focusedKind && !focusedKind.startsWith("panel:") && focusedKind !== "window-placeholder" ? focusedId : primaryId;
-    if (!id) return false;
-    const history = tree.activeTabHistory?.(id) || [];
-    if (history.length > 0) return true;
-    const leaf = tree.get(id);
-    const kind = leaf?.content?.kind;
+    const scope = this._backScope();
+    if (!scope) return false;
+    const content = this._scopeContent(scope);
+    if (typeof content?.canNavigateBack === "function") {
+      try {
+        if (content.canNavigateBack() === true) return true;
+      } catch (err) {
+        console.warn("[wm] content canNavigateBack threw", err);
+      }
+    }
+    if (scope.leafId) {
+      const tree = this._tree();
+      const history = tree.activeTabHistory?.(scope.leafId) || [];
+      if (history.length > 0) return true;
+      const kind2 = tree.get(scope.leafId)?.content?.kind;
+      return !!(kind2 && this.taxonomy.parentKindFor(kind2));
+    }
+    const kind = this._scopeActiveTab(scope)?.kind;
     return !!(kind && this.taxonomy.parentKindFor(kind));
   }
   /** Persist editor sub-state (e.g. active sub-tab) into the active
@@ -4276,6 +4316,25 @@ var WindowManager = class _WindowManager {
     const src = this._scopeTabs(scope);
     return src ? src.tabs[src.activeIdx] || null : null;
   }
+  /** C37. What the scope's content factory returned for what is on screen:
+   *  a window's `mountInfo`, or the tile's mounted content. */
+  _scopeContent(scope) {
+    if (scope?.windowId) return this._windowToLeaf.get(scope.windowId)?.mountInfo || null;
+    return scope?.leafId ? this.renderer?.contentOf?.(scope.leafId) || null : null;
+  }
+  /** C37. Ask the content to climb one of its own levels; `true` if it did.
+   *  A content that throws is treated as having nowhere to go, so Backspace
+   *  still walks the tile rather than doing nothing. */
+  _contentNavigateBack(scope) {
+    const content = this._scopeContent(scope);
+    if (typeof content?.navigateBack !== "function") return false;
+    try {
+      return content.navigateBack() === true;
+    } catch (err) {
+      console.warn("[wm] content navigateBack threw", err);
+      return false;
+    }
+  }
   /** Replace what a scope shows, in place: the tile's active tab (recording
    *  history) or the window's content. */
   _navigateInScope(scope, kind, props = {}) {
@@ -8008,6 +8067,7 @@ export {
   openTileTabMenu,
   openTileTabSwitcher,
   registerPanelKeys,
+  renderBreadcrumb,
   saveDesktops,
   uninstallPanelKeyRouter,
   wireLandingPaneFocus