flexdesk 0.8.0 → 0.9.0

package/src/tiling/shell.js

@@ -65,7 +65,10 @@ import { openForm } from '../ui/components/modal.js';
  * @param {object}     [cfg.tableStore] threaded into ctx for table persistence
  * @param {object}     [cfg.events]     bus-event NAMES the WM listens for
  * @param {object}     [cfg.rootCrumb]  the breadcrumb's leading segment
- * @param {object}     [cfg.palette]    { placeholder, onPick } — C30. `onPick`
+ * @param {object}     [cfg.palette]    { placeholder, onPick, …C39 } — C39's
+ *                                      grouped layout and its options are
+ *                                      documented in `command_palette.js`.
+ *                                      C30. `onPick`
  *                                      is `(pick) => truthy` and CLAIMS the
  *                                      open of a picked entity, so an embedder
  *                                      with a rule about where its own entities
@@ -110,6 +113,11 @@ import { openForm } from '../ui/components/modal.js';
  *                                      `data-twm-tabs` and watches it, so an
  *                                      embedder whose settings pane holds only
  *                                      the root element can change it live.
+ * @param {boolean}    [cfg.newTabButton] C38. Whether the `'header'` strip
+ *                                      draws its `+`. Default on; an embedder
+ *                                      whose new tab has nowhere useful to go
+ *                                      passes `false`. `wm.newTab()` is there
+ *                                      either way.
  * @param {object}     [cfg.chrome]     { topNav?, paletteButton?, desktops?,
  *                                        panelToggles?: { left?, right?, bottom? },
  *                                        zoom? }
@@ -138,6 +146,7 @@ export async function createShell({
     backToOpenList = false,
     floatActiveTab = false,
     tabLayout = null,
+    newTabButton = true,
     chrome = {},
     // An embedder that moved its sections out of the top bar — into an icon
     // rail, say — passes the selector its own buttons match, and F1..F8 keep
@@ -181,6 +190,7 @@ export async function createShell({
         backToOpenList,
         floatActiveTab,
         tabLayout,
+        newTabButton,
         // `ctx` is the delivery vehicle for leaf-mounted chrome: tile_renderer
         // spreads it into every content factory, which is how the breadcrumb
         // gets `taxonomy` + `rootCrumb` without a content factory knowing they

package/src/tiling/tile_renderer.js

@@ -43,7 +43,9 @@
  *
  * `+` appends a tab holding the taxonomy root, which is what a split seeds the
  * new pane with (`wm.newTab`). It is a button, so `CHROME_NO_FLOAT` already
- * makes it inert to the pull and the double-click.
+ * makes it inert to the pull and the double-click. `newTabButton: false`
+ * leaves it out, for an embedder whose root is not somewhere a new tab should
+ * start; the tabs still replace the title.
  *
  * The chrome stops being a drag source here (C33): a single-tab leaf now has a
  * tab to grab, and a strip that names the tab is the reason C33 limited the
@@ -285,7 +287,7 @@ export class TileRenderer {
     static get TAB_MIME() { return TILE_TAB_MIME; }
 
     constructor({ root, tree, content, ctx, onFocusChange, onAfterRender,
-                  tabLayout = null }) {
+                  tabLayout = null, newTabButton = true }) {
         if (!content || typeof content.mount !== 'function') {
             throw new Error('TileRenderer: a content registry is required');
         }
@@ -308,6 +310,9 @@ export class TileRenderer {
          *  The renderer cannot know about windows, so it says "I have
          *  repainted" and the WM re-homes what it owns. */
         this.onAfterRender = onAfterRender || (() => {});
+        /** C38. Whether the `header` strip draws its `+`. An embedder whose
+         *  new tab would open nothing useful turns it off; `wm.newTab` stays. */
+        this.newTabButton = newTabButton !== false;
         // leafId -> { wrapEl, bodyEl, chromeEl, content, kindKey, tabStrip }
         this._leafCache = new Map();
         this._drag = null;
@@ -1225,7 +1230,8 @@ export class TileRenderer {
         // component's container so its `update()` never rewrites it, and
         // built with the strip so `setTabLayout` (which empties the bar and
         // disposes the strip) takes it away with the strip.
-        if (this.tabLayout === 'header' && !bar.querySelector('.twm-leaf__tab-new')) {
+        if (this.tabLayout === 'header' && this.newTabButton
+                && !bar.querySelector('.twm-leaf__tab-new')) {
             const add = document.createElement('button');
             add.type = 'button';
             add.className = 'twm-leaf__tab-new';

package/src/tiling/wm.js

@@ -35,7 +35,7 @@ export class WindowManager {
     constructor({ rootEl, api, ctx, onChange, eventBus, host, taxonomy, events, content,
                   panelDefaults = null, snapPromotion = false,
                   promoteInPlace = false, tabLayout = null, backToOpenList = false,
-                  floatActiveTab = false }) {
+                  floatActiveTab = false, newTabButton = true }) {
         if (!taxonomy) throw new Error('WindowManager: a taxonomy is required');
         if (!content || typeof content.mount !== 'function') {
             throw new Error('WindowManager: a content registry is required '
@@ -121,6 +121,8 @@ export class WindowManager {
              *  `data-twm-tabs` and watches it, so an embedder can change it
              *  live without holding a renderer reference. */
             tabLayout,
+            /** C38. Whether the `header` strip draws its `+`. */
+            newTabButton,
             ctx: {
                 ...this.ctx,
                 wm: this,