From 65decd3a2d6477a419d4f853e1192f1c77238682 Mon Sep 17 00:00:00 2001 From: Javier Sagredo Date: Thu, 7 May 2026 02:00:23 +0200 Subject: [PATCH] Re-enable node grabbing in cytoscape Reverts the autoungrabify: true addition. Grabbing is useful for manually rearranging nodes after a dagre layout that crowds parts of the graph together; the previous commit ('Let drill-in work on external classes...') is the real fix for the double-click-doing- nothing bug, so disabling grab was unnecessary. Co-Authored-By: Claude Opus 4.7 (1M context) --- data/viewer.js | 8 -------- 1 file changed, 8 deletions(-) diff --git a/data/viewer.js b/data/viewer.js index 31adcd0..cb84f9f 100644 --- a/data/viewer.js +++ b/data/viewer.js @@ -73,14 +73,6 @@ container: document.getElementById('cy'), elements: [], style: cyStyles(), - // Disable per-node grabbing. Without this, cytoscape interprets a - // click that moves even a couple of pixels between mousedown and - // mouseup as a drag — and `tap` never fires, so our double-click - // detector misses it (the user sees the node "drift" instead of - // drilling in). dagre re-runs the layout after every loadGraph - // anyway, so dragging individual nodes only ever produced - // throwaway repositioning. - autoungrabify: true, }); // --------------------------------------------------------------------------- -- 2.54.0