feat(#332): Dashboard Command/Ctrl-drag tile move, text selection, shared cell-detail

[BorisTyshkevich]

What & why

Closes #332.

Improves Dashboard interaction consistency and data inspection, interaction-wiring only — no Dashboard schema, query execution, panel format, or persistence semantics change.

1. Command/Ctrl-drag tile movement. The native whole-card HTML5 draggable (and its dragstart/dragover/drop wiring) is removed. Tile reorder is now a modifier-gated pointer gesture that matches the schema graph (#55): a move starts only on primary button + Command/Ctrl + a 4px movement threshold. A plain drag never reorders and never preventDefaults an unmodified pointerdown, so text now selects and copies inside Markdown/text, table, and logs tiles. Hit-testing is a pure core/tile-reorder.ts helper over rects captured at drag-start (happy-dom's elementFromPoint always returns null, so hit-testing can't be a live DOM query). A completed move dispatches the existing atomic move-tile command exactly once and preserves canonical dashboard.tiles[] order; a cancelled gesture (pointercancel / window blur / Escape) changes nothing; the click a completed move synthesizes on its origin card is swallowed so it never activates a cell. While Command/Ctrl is held the grid shows a grab affordance (.dash-grid.modkey); an active move dims the moving card and outlines the drop target. Read-only Dashboards never wire movement.

2. Text selection inside tiles. Falls out of (1): no native drag, and user-select is disabled only for the duration of an active modifier-drag. Edit chrome (grip/delete/resize) stays non-selectable.

3–4. Shared cell-detail for table and logs tiles. The Dashboard onCell (previously a no-op) now calls the shared Workbench openCellDetail (results.ts) in the Dashboard's own document — in both edit and read-only modes — with the source column name/type and the raw (untruncated) value. No Dashboard-specific viewer is constructed. core/logs.ts logRowDisplay is widened to {name,type,raw,display} per field so each source-backed log field (time, level, message, and each extra) is independently clickable and keyboard-operable (Enter/Space); a selection gesture ending on a field never opens the drawer. DashboardApp gains prefs for the shared drawer's resize-persist seam.

Layering

• Pure logic in src/core/: new tile-reorder.ts (threshold + rect hit-test) and the widened logs.ts row-shaping, both 100% covered.
• src/ui/dashboard.ts stays a thin adapter over the pure helpers; the library-free pointer wiring is fully unit-tested under happy-dom.

Review

A self-review pass caught and fixed an inverted click-suppression (a same-tile ⌘-drag would have spuriously opened cell-detail) and a vacuous suppression test; both are now covered. Added guards against overlapping gestures and a flow-KPI tile's unplaced card being a phantom drop target.

Checklist

• npm test passes (per-file coverage gate)
• Tests added/updated in the same change as the code
• npm run build succeeds (single-file dist/sql.html)
• Layers kept honest: pure logic in src/core/, DOM in src/ui/
• No new runtime dependency
• CHANGELOG.md ([Unreleased]) updated
• Reconciled affected tracked work — the Dashboard v1 phase 4 (#280): DashboardViewerSession + normative flow@1 layout #286 "pointer-drag-only" note in roadmap Roadmap to 1.0.0 #68 now reflects the Command/Ctrl-drag change

Test plan

• Unit gate green (156 files, 4308+ tests); real-browser verification (text selection, ⌘/Ctrl-drag + destination feedback, drawer resize/backdrop, light/dark, Grid Tiles/Full/Report/2-col/3-col, detached read-only route, narrow viewport) is done via e2e (GitHub Actions) + a manual otel-cluster pass — e2e is CI-only in this environment, not run locally.

🤖 Generated with Claude Code

[BorisTyshkevich]

Follow-up (commit b613f78): reworked the tile-move gesture per owner feedback — the modifier-only drag with a drop-target outline is replaced by grip-drag (no modifier) / ⌘-Ctrl body-drag + live grafana-grid reflow (dragged tile lifts and follows, siblings shift to open a gap) with a ≥2/3-overlap commit, else snap back. Snap-back restore is synchronous (independent of the signature-gated reconcile). Flow engine keeps the simpler point-hit-test drag. An adversarial review caught a forward-drag preview/commit off-by-one (placeholder previewed one slot earlier than the splice landed) — fixed so the placeholder previews the exact committed slot, with a regression test. Gate + build green; CHANGELOG and #68 reconciled.

[BorisTyshkevich]

Two more refinements (commit 25cb573), all CI green (test/e2e ×2/bundle/docker/size):

• ≥2/3-overlap commit now measures the TARGET slot's area, not the dragged tile's — correct for tiles of differing span/height (a small tile dropped inside a big slot no longer commits just by being contained; it must cover 2/3 of the slot).
• Markdown cell-detail preview — a table/log cell value that looks like Markdown opens the drawer on a Rendered↔Source toggle (like HTML) using the docs pane's top-quality renderDocMarkdown viewer (bounded, fail-closed: no innerHTML, links via the doc link policy, code fences fall back to plain <pre>). Plain non-markup text still shows source only.

Verified live on otel (build 25cb573): a Markdown cell renders # Heading→h4 + list + blockquote with a working Source toggle; no e2e harness imports results.ts, so the new marked-backed import adds no raw-ESM bare-import dependency. Coverage gates hold (core/cell.ts + core/tile-reorder.ts 100%; results.ts/dashboard.ts within gate).

[BorisTyshkevich]

Fixed the reported issue (clicking the Markdown tile did nothing) — commit c4553c6, all CI green (test/e2e ×2/bundle/docker/size):

• The Dashboard Text (Markdown) tile now renders inline through the same top-quality doc viewer as the reference docs (retiring the lighter markdown-lite renderer — the app now has ONE Markdown paradigm).
• The Text tile is click / Enter-Space openable into the shared preview drawer (full Markdown, resizable), with a selection-guard and inner-link guard so a drag-select or a link click never opens it.

Verified live on otel (c4553c6): the "About this demo" tile renders .md-view > .docs-md inline and clicking it opens the drawer with the rendered Markdown. Text-panel link policy is panel-appropriate (http(s) absolute pass; scheme-less → literal text, no clickhouse.com rewrite). No e2e harness imports panels.ts/results.ts, so the doc-markdown (marked) graph reaches no raw-ESM harness. Gate/build/arch clean.