From 1b06a2f46b21a44e556a2e32763681cd700dcdec Mon Sep 17 00:00:00 2001 From: juan Date: Mon, 23 Mar 2026 00:00:38 -0700 Subject: [PATCH] Revert "Use nearest-edge distance for symmetrical drag reorder of tall sections" This reverts commit 53c5f5759d0db0b5358c115f560da18dec9c6455. --- .../controllers/dashboard_sortable_controller.js | 13 ++----------- 1 file changed, 2 insertions(+), 11 deletions(-) diff --git a/app/javascript/controllers/dashboard_sortable_controller.js b/app/javascript/controllers/dashboard_sortable_controller.js index c1ab01780..c0a0cbcca 100644 --- a/app/javascript/controllers/dashboard_sortable_controller.js +++ b/app/javascript/controllers/dashboard_sortable_controller.js @@ -295,19 +295,10 @@ export default class extends Controller { draggableElements.forEach((child) => { const rect = child.getBoundingClientRect(); const centerX = rect.left + rect.width / 2; + const centerY = rect.top + rect.height / 2; - // Use distance to nearest edge instead of center so tall sections - // don't require more finger travel than short ones const dx = pointerX - centerX; - let dy; - if (pointerY < rect.top) { - dy = rect.top - pointerY; - } else if (pointerY > rect.bottom) { - dy = pointerY - rect.bottom; - } else { - dy = 0; // pointer is within this section's vertical bounds - } - + const dy = pointerY - centerY; const distance = Math.sqrt(dx * dx + dy * dy); if (distance < minDistance) {