From abf1d73aa6932275540b7b47cd2e93d39591cccb Mon Sep 17 00:00:00 2001
From: Orion Reed <orionreedt@gmail.com>
Date: Sun, 15 Dec 2024 05:43:50 -0500
Subject: [PATCH] default shape size

---
 src/folk-toolset.ts | 19 ++++++++++++-------
 1 file changed, 12 insertions(+), 7 deletions(-)

diff --git a/src/folk-toolset.ts b/src/folk-toolset.ts
index 643fe97..b7d34de 100644
--- a/src/folk-toolset.ts
+++ b/src/folk-toolset.ts
@@ -55,7 +55,7 @@ export class FolkShapeTool extends FolkInteractionHandler {
 
     switch (event.type) {
       case 'pointerdown':
-        if (!target || target instanceof FolkShape) return;
+        if (!target || target instanceof FolkShape || target instanceof FolkInteractionHandler) return;
         event.stopImmediatePropagation();
 
         const rect = target.getBoundingClientRect();
@@ -106,13 +106,18 @@ export class FolkShapeTool extends FolkInteractionHandler {
         if (!this.currentShape) return;
         event.stopImmediatePropagation();
 
-        // Remove shape if it's too small
-        if (this.currentShape.width < 10 || this.currentShape.height < 10) {
-          this.currentShape.remove();
-        } else {
-          this.currentShape.focus();
+        // If the shape is too small (meaning almost no drag occurred)
+        // create a default sized shape instead
+        if (this.currentShape.width <= 1 || this.currentShape.height <= 1) {
+          const defaultSize = 100;
+          this.currentShape.width = defaultSize;
+          this.currentShape.height = defaultSize;
+          // Center the shape on the click point
+          this.currentShape.x = this.startPoint!.x - defaultSize / 2;
+          this.currentShape.y = this.startPoint!.y - defaultSize / 2;
         }
 
+        this.currentShape.focus();
         target.releasePointerCapture(event.pointerId);
         this.currentShape = null;
         this.startPoint = null;
@@ -182,7 +187,7 @@ export class FolkToolset extends HTMLElement {
     }
 
     // If clicking same tool, just deactivate
-    if (this.currentHandler === tool.handleEvent.bind(tool)) {
+    if (this.activeTool === tool) {
       this.currentHandler = null;
       this.activeTool = null;
       return;