flexdesk 0.16.1 → 0.17.1

package/src/tiling/command_palette.js

@@ -65,7 +65,10 @@
  *   groupLabel the group's header, when the group is not a taxonomy kind
  *   kindLabel  the word drawn on the right, when the taxonomy's is wrong
  *   boost      added to the match score (an embedder's "recently opened")
- *   modes      which of 'split' / 'window' the row offers (key hints)
+ *   modes      which of 'split' / 'window' the row offers (key hints). For
+ *              a row whose kind is single (KindDef `single`) both are copy
+ *              verbs: the hints are drawn greyed with the taxonomy's reason,
+ *              and Ctrl/Shift+Enter on the row does nothing.
  *   drillLabel the Tab hint ("search rows"); Tab is offered only with it
  *   shortcut   a key binding drawn on the right
  *   verb       the Enter hint ('open' by default, 'run' for a command)
@@ -79,16 +82,33 @@
  */
 
 import { HelpModal } from '../help/help_modal.js';
+import { KEYBOARD_SHORTCUTS_TOPIC_ID, KEYMAP_SHORTCUTS, keyboardShortcutsTopic } from './keymap.js';
 
 const ROOT_ID = 'twm-cmdpal';
 
-// Non-entity commands surfaced in the palette's search results. They
-// carry an `action` so `commit()` runs them instead of opening a tile.
-const STATIC_COMMANDS = [
-    { kind: 'command', id: 'shortcuts', label: 'Keyboard shortcuts',
-      hint: 'Show all key bindings (?)', icon: 'keyboard', shortcut: '?', verb: 'run',
-      action: () => HelpModal.open('keyboard-shortcuts') },
-];
+/** The words a shortcuts table uses for the help action — the default `?` row's. */
+const HELP_ACTION = KEYMAP_SHORTCUTS.find((r) => r.keys === '?').action;
+
+/**
+ * Non-entity commands surfaced in the palette's search results. They carry an
+ * `action` so `commit()` runs them instead of opening a tile. `shortcuts` is
+ * the table the *Keyboard shortcuts* command shows — the shell passes the one
+ * its keymap's bindings produced (`keymapShortcuts`). The command's hint names
+ * the keys that open help in that table: `?` by default, `F9` when an embedder
+ * moved help there, none when no key opens it.
+ */
+function _staticCommands(shortcuts) {
+    const helpKeys = shortcuts.filter((r) => r.action === HELP_ACTION).map((r) => r.keys).join(' / ');
+    return [
+        { kind: 'command', id: 'shortcuts', label: 'Keyboard shortcuts',
+          hint: helpKeys ? `Show all key bindings (${helpKeys})` : 'Show all key bindings',
+          icon: 'keyboard', ...(helpKeys ? { shortcut: helpKeys } : {}), verb: 'run',
+          // The application's help may define the topic; when it does not, the
+          // window manager's own bindings are shown rather than an error.
+          action: () => HelpModal.open(KEYBOARD_SHORTCUTS_TOPIC_ID,
+                                       { fallback: keyboardShortcutsTopic(shortcuts) }) },
+    ];
+}
 
 const DEFAULT_GROUP_CAP = 4;
 const DEFAULT_COMMAND_MIN_CHARS = 2;
@@ -104,7 +124,9 @@ const MAX_ROWS = 200;
  *                     gesture. See `commit` for what it is for; omitted, the
  *                     palette behaves as it always has.
  *  @param chrome, layout, groupCap, empty, emptyLabel, commands,
- *         commandMinChars, drill, cache, tips, footerNote — C39, above. */
+ *         commandMinChars, drill, cache, tips, footerNote — C39, above.
+ *  @param shortcuts   the rows the built-in *Keyboard shortcuts* command lists
+ *                     (`{keys, action}`); `KEYMAP_SHORTCUTS` when omitted. */
 export function createCommandPalette({ wm, api, taxonomy, catalog, placeholder = 'Search…',
                                        onPick = null,
                                        chrome = {}, layout = 'flat',
@@ -113,7 +135,8 @@ export function createCommandPalette({ wm, api, taxonomy, catalog, placeholder =
                                        commands = null,
                                        commandMinChars = DEFAULT_COMMAND_MIN_CHARS,
                                        drill = null, cache = false,
-                                       tips = null, footerNote = '' }) {
+                                       tips = null, footerNote = '',
+                                       shortcuts = KEYMAP_SHORTCUTS }) {
     if (!taxonomy) throw new Error('createCommandPalette: a taxonomy is required');
     if (!catalog)  throw new Error('createCommandPalette: an entity catalog is required');
     const grouped = layout === 'grouped';
@@ -122,6 +145,9 @@ export function createCommandPalette({ wm, api, taxonomy, catalog, placeholder =
     let cached = null;            // C39 `cache`: the last loaded catalogue
     let onClose = null;
 
+    /** A row whose kind exists once (KindDef `single`): no split, no window. */
+    const isSingle = (row) => !!row && taxonomy.isSingle(row.kind);
+
     const isOpen = () => !!overlay;
     const close = () => {
         if (!overlay) return;
@@ -132,6 +158,7 @@ export function createCommandPalette({ wm, api, taxonomy, catalog, placeholder =
     };
     const toggle = () => { isOpen() ? close() : open(); };
     const invalidate = () => { cached = null; };
+    const staticCommands = _staticCommands(shortcuts);
 
     const open = async () => {
         if (overlay) return;
@@ -152,11 +179,22 @@ export function createCommandPalette({ wm, api, taxonomy, catalog, placeholder =
         const chips = overlay.querySelectorAll('[data-shortcut]');
         const toggles = overlay.querySelectorAll('[data-toggle]');
 
-        // Reflect current panel state in toggle buttons.
-        for (const btn of toggles) {
+        // Reflect current panel state in toggle buttons. A panel the primary
+        // content withholds is a chip that says why and does nothing.
+        const syncToggle = (btn) => {
             const which = btn.dataset.toggle;
+            const { available, reason } = wm.getPanelAvailability(which);
             btn.classList.toggle('twm-chip--on', wm.isPanelOpen(which));
-        }
+            if (available) {
+                btn.removeAttribute('aria-disabled');
+                btn.removeAttribute('title');
+            } else {
+                btn.setAttribute('aria-disabled', 'true');
+                if (reason) btn.setAttribute('title', reason);
+                else btn.removeAttribute('title');
+            }
+        };
+        toggles.forEach(syncToggle);
 
         chips.forEach((btn) => btn.addEventListener('click', () => {
             const kind = btn.dataset.shortcut;
@@ -164,9 +202,8 @@ export function createCommandPalette({ wm, api, taxonomy, catalog, placeholder =
             close();
         }));
         toggles.forEach((btn) => btn.addEventListener('click', () => {
-            const which = btn.dataset.toggle;
-            wm.togglePanel(which);
-            btn.classList.toggle('twm-chip--on', wm.isPanelOpen(which));
+            wm.togglePanel(btn.dataset.toggle);
+            syncToggle(btn);
         }));
 
         // Scoped-search state. Typing `<prefix>:` converts the prefix
@@ -296,8 +333,13 @@ export function createCommandPalette({ wm, api, taxonomy, catalog, placeholder =
             const modes = Array.isArray(r.modes) ? r.modes : [];
             const acts = [`<span><kbd>Enter</kbd> ${_esc(verb)}</span>`];
             if (r.drillLabel && drill) acts.push(`<span><kbd>Tab</kbd> ${_esc(r.drillLabel)}</span>`);
-            if (modes.includes('split'))  acts.push('<span><kbd>Ctrl Enter</kbd> split</span>');
-            if (modes.includes('window')) acts.push('<span><kbd>Shift Enter</kbd> window</span>');
+            // A single kind has no copies: its split and window hints are
+            // drawn refused, with the reason, rather than offered.
+            const refused = isSingle(r)
+                ? ` class="twm-cmdpal__act--disabled" aria-disabled="true" title="${_esc(taxonomy.singleReason(r.kind))}"`
+                : '';
+            if (modes.includes('split'))  acts.push(`<span${refused}><kbd>Ctrl Enter</kbd> split</span>`);
+            if (modes.includes('window')) acts.push(`<span${refused}><kbd>Shift Enter</kbd> window</span>`);
             const side = r.shortcut
                 ? `<kbd class="twm-cmdpal__shortcut">${_esc(r.shortcut)}</kbd>`
                 : `<span class="twm-cmdpal__kind">${_esc(kindLabel(r))}</span>`;
@@ -411,6 +453,9 @@ export function createCommandPalette({ wm, api, taxonomy, catalog, placeholder =
             const picked = filtered[i];
             if (!picked) return;
             if (picked.more) { showAll(picked.more); return; }
+            // Split and window open a COPY; a single kind refuses both, and the
+            // palette stays open on the row whose hints say why.
+            if (mode !== 'open' && isSingle(picked)) return;
             const pick = _public(picked);
             const how = { mode, scope: scope?.id ?? null };
             close();
@@ -668,7 +713,7 @@ export function createCommandPalette({ wm, api, taxonomy, catalog, placeholder =
                 try { extra = (commands() || []).map((c) => ({ kind: 'command', ...c })); }
                 catch (err) { console.warn('[cmdpal] commands() threw', err); }
             }
-            return [...STATIC_COMMANDS, ...extra];
+            return [...staticCommands, ...extra];
         };
         // Objects before commands in the grouped layout, so that a tie that
         // survives every rule goes to the object (first seen wins).

package/src/tiling/content_registry.js

@@ -12,7 +12,10 @@
  * `ctx` is the shell's ctx plus, per leaf, `leafId` and `setTitle(text)` —
  * rename the pane's chrome title from inside the content, at mount or any
  * time after; it survives repaints, and `setTitle(null)` gives the title back
- * to the tree (tile_renderer.js, C41).
+ * to the tree (tile_renderer.js, C41) — and `setChromeActions(specs)`, which
+ * replaces the pane's content buttons the same way: at mount (winning over a
+ * returned `chromeActions`) or any time after, surviving repaints, `null` or
+ * `[]` clearing them (tile_renderer.js, C42).
  *
  * ── ONE REGISTRY PER SHELL ────────────────────────────────────────────
  * This module used to hold a module-level `Map` plus a `register(kind, fn)`

package/src/tiling/desktops.js

@@ -4,7 +4,8 @@
  * Where that key actually lands is the embedder's business (see the host's
  * `resolvePath`); this module only names it.
  *
- * Each desktop = { id, label, tree (TileTree), windows ([]), panels }.
+ * Each desktop = { id, label, tree (TileTree), windows ([]), panels,
+ * panelShares }.
  * Switching desktops swaps the active tree under the renderer.
  */
 
@@ -47,6 +48,11 @@ function _makeDesktop(label, seed, panelDefaults = DEFAULT_PANEL_STATE) {
         // when it asked for nothing. Names are assigned by wm._canonicalize via
         // PANEL_TITLES.
         panels: { ...panelDefaults },
+        // Each panel's last share of the WM area along its axis, recorded by
+        // the WM when a splitter drag ends (and, for a panel not yet
+        // measured, when it rebuilds the panels). A side with none opens at
+        // the default ratio.
+        panelShares: {},
         // C40. Both unset is a desktop exactly as it always was.
         group: null,
         fixed: false,
@@ -159,6 +165,9 @@ export class DesktopManager {
                 id: d.id, label: d.label,
                 tree: d.tree.serialize(),
                 panels: { ...d.panels },
+                // Written only when a panel has been measured, like the C40
+                // keys below.
+                ...(Object.keys(d.panelShares).length ? { panelShares: { ...d.panelShares } } : {}),
                 // C40. Written only when set, so a blob without them is
                 // byte-for-byte the blob it always was.
                 ...(d.group ? { group: d.group } : {}),
@@ -182,6 +191,7 @@ export class DesktopManager {
             // this merge order avoids. The default only fills a key the stored
             // blob predates.
             panels: { ...m.panelDefaults, ...(raw.panels || {}) },
+            panelShares: { ...(raw.panelShares || {}) },
             group: raw.group || null,
             fixed: !!raw.fixed,
         }));