From 0d6717896e87dd20f8bcf19a6ba4464aa44f52ff Mon Sep 17 00:00:00 2001
From: Emma <ironicallyshort737@hotmail.com>
Date: Tue, 7 Feb 2023 16:09:45 -0700
Subject: [PATCH] fixed issue of being able to add a point to draggables twice

---
 bundle.js | 4 +---
 canvas.ts | 5 +----
 2 files changed, 2 insertions(+), 7 deletions(-)

diff --git a/bundle.js b/bundle.js
index c418ceb..503e180 100644
--- a/bundle.js
+++ b/bundle.js
@@ -390,6 +390,7 @@ class Doodler {
     mouseX = 0;
     mouseY = 0;
     registerDraggable(point, radius, style) {
+        if (this.draggables.find((d)=>d.point === point)) return;
         const id = this.addUIElement('circle', point, radius, {
             fillColor: '#5533ff50',
             strokeColor: '#5533ff50'
@@ -410,9 +411,6 @@ class Doodler {
         this.draggables = this.draggables.filter((d)=>d.point !== point);
     }
     onClick(e) {
-        const rect = this._canvas.getBoundingClientRect();
-        e.clientX - rect.left;
-        e.clientY - rect.top;
         for (const d of this.draggables){
             if (d.point.dist(new Vector(this.mouseX, this.mouseY)) <= d.radius) {
                 d.beingDragged = true;
diff --git a/canvas.ts b/canvas.ts
index 93cdf26..1130c71 100644
--- a/canvas.ts
+++ b/canvas.ts
@@ -209,6 +209,7 @@ export class Doodler {
 
 
   registerDraggable(point: Vector, radius: number, style?: IStyle & { shape: 'square' | 'circle' }) {
+    if (this.draggables.find(d => d.point === point)) return;
     const id = this.addUIElement('circle', point, radius, { fillColor: '#5533ff50', strokeColor: '#5533ff50' })
     this.draggables.push({ point, radius, style, id });
   }
@@ -222,10 +223,6 @@ export class Doodler {
   }
 
   onClick(e: MouseEvent) {
-    const rect = this._canvas.getBoundingClientRect();
-    const mouseX = e.clientX - rect.left;
-    const mouseY = e.clientY - rect.top;
-
     for (const d of this.draggables) {
       if (d.point.dist(new Vector(this.mouseX, this.mouseY)) <= d.radius) {
         d.beingDragged = true;