flexdesk 0.4.7 → 0.5.0

package/package.json

@@ -1,81 +1,81 @@
-{
-    "name": "flexdesk",
-    "version": "0.4.7",
-    "description": "A tiling window manager and widget set for desktop-class web applications. Zero runtime dependencies.",
-    "license": "MIT",
-    "repository": {
-        "type": "git",
-        "url": "git+ssh://git@github.com/nor-os/FlexDesk.git"
-    },
-    "homepage": "https://github.com/nor-os/FlexDesk",
-    "bugs": {
-        "url": "https://github.com/nor-os/FlexDesk/issues"
-    },
-    "type": "module",
-    "sideEffects": [
-        "*.css"
-    ],
-    "files": [
-        "dist",
-        "src",
-        "css",
-        "README.md",
-        "LICENSE",
-        "THIRD_PARTY_LICENSES.md"
-    ],
-    "exports": {
-        "./core": {
-            "import": "./dist/core.js",
-            "development": "./core.js"
-        },
-        "./host": {
-            "import": "./dist/host.js",
-            "development": "./host.js"
-        },
-        "./wm": {
-            "import": "./dist/wm.js",
-            "development": "./wm.js"
-        },
-        "./widgets": {
-            "import": "./dist/widgets.js",
-            "development": "./widgets.js"
-        },
-        "./css": "./dist/flexdesk.css",
-        "./css/tokens": "./css/tokens.css",
-        "./charts": {
-            "import": "./dist/charts.js",
-            "development": "./charts.js"
-        },
-        "./tiles": {
-            "import": "./dist/tiles.js",
-            "development": "./tiles.js"
-        },
-        "./editor": {
-            "import": "./dist/editor.js",
-            "development": "./editor.js"
-        }
-    },
-    "scripts": {
-        "build": "node build.mjs",
-        "check": "node build.mjs --check",
-        "test": "node tests/run.mjs"
-    },
-    "devDependencies": {
-        "esbuild": "^0.28.1"
-    },
-    "peerDependenciesMeta": {
-        "plotly.js": {
-            "optional": true
-        },
-        "monaco-editor": {
-            "optional": true
-        }
-    },
-    "keywords": [
-        "tiling",
-        "window-manager",
-        "datatable",
-        "command-palette",
-        "ui"
-    ]
-}
+{
+    "name": "flexdesk",
+    "version": "0.5.0",
+    "description": "A tiling window manager and widget set for desktop-class web applications. Zero runtime dependencies.",
+    "license": "MIT",
+    "repository": {
+        "type": "git",
+        "url": "git+ssh://git@github.com/nor-os/FlexDesk.git"
+    },
+    "homepage": "https://github.com/nor-os/FlexDesk",
+    "bugs": {
+        "url": "https://github.com/nor-os/FlexDesk/issues"
+    },
+    "type": "module",
+    "sideEffects": [
+        "*.css"
+    ],
+    "files": [
+        "dist",
+        "src",
+        "css",
+        "README.md",
+        "LICENSE",
+        "THIRD_PARTY_LICENSES.md"
+    ],
+    "exports": {
+        "./core": {
+            "import": "./dist/core.js",
+            "development": "./core.js"
+        },
+        "./host": {
+            "import": "./dist/host.js",
+            "development": "./host.js"
+        },
+        "./wm": {
+            "import": "./dist/wm.js",
+            "development": "./wm.js"
+        },
+        "./widgets": {
+            "import": "./dist/widgets.js",
+            "development": "./widgets.js"
+        },
+        "./css": "./dist/flexdesk.css",
+        "./css/tokens": "./css/tokens.css",
+        "./charts": {
+            "import": "./dist/charts.js",
+            "development": "./charts.js"
+        },
+        "./tiles": {
+            "import": "./dist/tiles.js",
+            "development": "./tiles.js"
+        },
+        "./editor": {
+            "import": "./dist/editor.js",
+            "development": "./editor.js"
+        }
+    },
+    "scripts": {
+        "build": "node build.mjs",
+        "check": "node build.mjs --check",
+        "test": "node tests/run.mjs"
+    },
+    "devDependencies": {
+        "esbuild": "^0.28.1"
+    },
+    "peerDependenciesMeta": {
+        "plotly.js": {
+            "optional": true
+        },
+        "monaco-editor": {
+            "optional": true
+        }
+    },
+    "keywords": [
+        "tiling",
+        "window-manager",
+        "datatable",
+        "command-palette",
+        "ui"
+    ]
+}

package/src/tiling/landing_table.js

@@ -3,7 +3,10 @@
  * pattern used by SFC / Markets / Scenarios / KPIs landings (and the
  * market-archetype instances table). Three responsibilities:
  *
- *   1. Single-click row → open the entity (handled by callers).
+ *   1. Single-click row → open the entity (handled by callers). Since 0.5.0
+ *      the click is judged by the same rule as `DataTable`'s `onRowActivate`
+ *      (`ui/components/row_activation.js`); a table that owns its rows can
+ *      use that option directly instead of this helper.
  *   2. Right-click row → custom context menu with Open / Edit / Delete
  *      (replaces DataTable's default copy-as-TSV menu for these tables;
  *      the default still applies to data tables outside this helper).
@@ -16,6 +19,7 @@
  */
 
 import { showContextMenu } from '../ui/components/context_menu.js';
+import { isRowActivation } from '../ui/components/row_activation.js';
 import { registerPanelKeys } from './panel_keys.js';
 
 /** Attach the navigation-table interactions to a host element that
@@ -85,10 +89,14 @@ export function attachLandingTableBehavior(host, getRow, entityActions, options
             dispatch(actionBtn.dataset.twmAction, tr ? getRow(tr.__rowIndex) : null);
             return;
         }
-        // Plain row click → open. Ignore clicks on interactive controls
-        // (buttons, filter inputs, links) and anything outside a body row
-        // (pagination strip, header, empty space).
-        if (ev.target.closest('button, a, input, select, textarea')) return;
+        // Plain row click → open — by THE row-activation rule DataTable's own
+        // `onRowActivate` uses (0.5.0, `ui/components/row_activation.js`), so
+        // the two cannot drift: not on a control (a button, a field, a link),
+        // not on the click that ends a drag selecting a cell's text, and not on
+        // the second click of a double-click, which used to open the row twice.
+        // Anything outside a body row (pagination strip, header, empty space)
+        // is not a row either.
+        if (!isRowActivation(ev, 'click', { scope: host })) return;
         const tr = bodyRowFor(ev);
         if (tr) dispatch('open', getRow(tr.__rowIndex));
     };