From e25db413c595623b18315bd716ba7171f32ccc51 Mon Sep 17 00:00:00 2001
From: Hsieh Chin Fan <pham@topo.tw>
Date: Sun, 15 Sep 2024 00:16:02 +0800
Subject: feat: Improve draggable behavior in overlay layout

* Set position=absolute on each block, so they won't affect each others'
  y value when display=none (about Block Formatting Context)
* Add title tooltip for removing block
* Simple logic to set initial position of block by their width
---
 src/css/dumbymap.css |  1 +
 src/dumbymap.mjs     | 60 ++++++++++++++++++++++++++++++++++------------------
 2 files changed, 40 insertions(+), 21 deletions(-)

(limited to 'src')

diff --git a/src/css/dumbymap.css b/src/css/dumbymap.css
index 1dc7a09..8e28af5 100644
--- a/src/css/dumbymap.css
+++ b/src/css/dumbymap.css
@@ -142,6 +142,7 @@
 
     > .draggable-block {
       box-sizing: content-box;
+      position: absolute;
       width: fit-content;
       max-height: 50vh;
       overflow: scroll;
diff --git a/src/dumbymap.mjs b/src/dumbymap.mjs
index f9a999b..4d4628d 100644
--- a/src/dumbymap.mjs
+++ b/src/dumbymap.mjs
@@ -294,6 +294,26 @@ export const generateMaps = async (container) => {
   console.info(renderInfo)
 
   //}}}
+  // Draggable Blocks{{{
+  // Add draggable part for blocks
+  htmlHolder.blocks = Array.from(htmlHolder.querySelectorAll('.draggable-block'))
+  htmlHolder.blocks.forEach(block => {
+    // Add draggable part
+    const draggablePart = document.createElement('div');
+    draggablePart.classList.add('draggable')
+    draggablePart.textContent = '☰'
+    draggablePart.title = 'Use middle-click to remove block'
+    block.insertBefore(draggablePart, block.firstChild)
+    block.draggablePart = draggablePart
+
+    draggablePart.onmouseup = (e) => {
+      if (e.button === 1) {
+        block.style.display = "none";
+      }
+    }
+  })
+
+  // }}}
   // CSS observer {{{
 
   // Set focusArea
@@ -323,20 +343,6 @@ export const generateMaps = async (container) => {
     }
   }
 
-  // Add draggable part for blocks
-  htmlHolder.blocks = Array.from(htmlHolder.querySelectorAll('.draggable-block'))
-  htmlHolder.blocks.forEach(block => {
-    const draggablePart = document.createElement('div');
-    draggablePart.classList.add('draggable')
-    draggablePart.textContent = '☰'
-
-    // TODO Better way to close block
-    draggablePart.onmouseup = (e) => {
-      if (e.button === 1) block.style.display = "none";
-    }
-    block.insertBefore(draggablePart, block.firstChild)
-  })
-
   // observe layout change
   const layoutObserver = new MutationObserver(() => {
     const layout = container.getAttribute('data-layout')
@@ -363,17 +369,29 @@ export const generateMaps = async (container) => {
     }
 
     if (layout === 'overlay') {
-      htmlHolder.blocks.forEach(block => {
-        block.draggableInstance = new PlainDraggable(block, { handle: block.querySelector('.draggable') })
-        block.draggableInstance.snap = { x: { step: 20 }, y: { step: 20 } }
-        // block.draggableInstance.onDragEnd = () => {
-        //   links(block).forEach(link => link.line.position())
-        // }
+      let x = 0;
+      let y = 0;
+      htmlHolder.blocks.forEach(block =>{
+        block.draggableInstance = new PlainDraggable(block, {
+          handle: block.draggablePart,
+          snap: { x: { step: 20 }, y: { step: 20 } },
+          autoScroll: false,
+        })
+        // block.style.transform = `translate(${x}px, ${y}px)`
+        block.style.left = `${x}px`
+        block.style.top = `${y}px`
+        x += parseInt(window.getComputedStyle(block).width) + 50
+        if (x > window.innerWidth) {
+          y += 200
+          x = x % window.innerWidth
+        }
       })
     } else {
       htmlHolder.blocks.forEach(block => {
         block.style.transform = 'none'
-        block.draggableInstance?.remove()
+        try {
+          block.draggableInstance?.remove()
+        } catch (err) { }
       })
     }
   });
-- 
cgit v1.2.3-70-g09d2