From 209cff1cfb58278a0d7bf34f72256e0e980bcf6d Mon Sep 17 00:00:00 2001
From: Stephan Egli <stephan.egli93@gmail.com>
Date: Fri, 10 Jan 2025 18:08:25 +0100
Subject: [PATCH] removed debug messages

---
 src/canvasManager.js | 11 -----------
 src/docManager.js    |  2 --
 src/main.js          | 27 +--------------------------
 3 files changed, 1 insertion(+), 39 deletions(-)

diff --git a/src/canvasManager.js b/src/canvasManager.js
index d99e4da..c65e577 100644
--- a/src/canvasManager.js
+++ b/src/canvasManager.js
@@ -12,16 +12,12 @@ const SVG_NS = "http://www.w3.org/2000/svg";
  * @param {Array} patches 
  */
 export function drawAllShapesOnCanvas(container, shapesArray, patches = []) {
-    console.log("drawAllShapesOnCanvas container:",container)
     if (!container || !(container instanceof SVGElement)) {
         return;
     }
 
-    console.log("Received patches:", patches);
-
     // Ensure patches is an array
     const patchArray = patches?.patches || patches || [];
-    console.log("patchArray:",patchArray)
     if (!Array.isArray(patchArray)) {
         console.warn("Expected patches to be an array, got:", typeof patchArray, patchArray);
         // Fall back to full redraw
@@ -56,21 +52,15 @@ export function drawAllShapesOnCanvas(container, shapesArray, patches = []) {
 
     // Process patches
     patchArray.forEach(patch => {
-        console.log("patch:",patch)
         // Check if this patch affects the shapes array
         if (patch.path[0] === 'shapes') {
-            console.log("patch.path[0] shapes:",patch.path[0])
             if (patch.action === 'put') {
-                // New shape added or shape updated
-                console.log("patch.path[1] shapeindex:",patch.path[1])
                 const shapeIndex = patch.path[1];
                 const shape = shapesArray[shapeIndex];
-                console.log("shape:",shape)
                 if (!shape) return;
 
                 let element = document.getElementById(shape.id);
                 if (!element) {
-                    console.log("create new element, shape:",shape)
                     // Create new element
                     if (shape.type === "circle") {
                         element = document.createElementNS(SVG_NS, "circle");
@@ -86,7 +76,6 @@ export function drawAllShapesOnCanvas(container, shapesArray, patches = []) {
                     }
                 } else {
                     // Update existing element
-                    console.log("update existing element, shape:",shape)
                     if (shape.type === "circle") {
                         updateCircleAttributes(element, shape);
                     } else if (shape.type === "triangle") {
diff --git a/src/docManager.js b/src/docManager.js
index 9be62a1..4e0095f 100644
--- a/src/docManager.js
+++ b/src/docManager.js
@@ -82,9 +82,7 @@ export function moveShape(shapeId, newX, newY) {
     // Find the index of the shape instead of using find()
     // TODO faster implementation than for loop
     for (let i = 0; i < doc.shapes.length; i++) {
-      console.log("moveShape doc.shapes[i]:",i,doc.shapes[i].id,Number(shapeId))
       if (doc.shapes[i].id === Number(shapeId)) {
-        console.log("moveShape doc.shapes[i]:",i,doc.shapes[i].x,newX,doc.shapes[i].y,newY)
         doc.shapes[i].x = newX;
         doc.shapes[i].y = newY;
         break;
diff --git a/src/main.js b/src/main.js
index 3cb7a6c..4839481 100644
--- a/src/main.js
+++ b/src/main.js
@@ -30,13 +30,9 @@ drawAllShapesOnCanvas(container, handle.docSync().shapes)
 
 // 2. Subscribe so we can re-render whenever doc changes
 handle.on("change", (change) => {
-  console.log("Document changed! Received patches:", change);
-  console.log("Current shapes:", change.doc.shapes);
   // Only redraw if we're not currently dragging
   // TODO if (!isDragging) {
-  // TODO modify name of the subsequent function:
   drawAllShapesOnCanvas(container, change.doc.shapes, change.patches);
-  
 });
 
 // 3. Buttons holen
@@ -60,31 +56,12 @@ container.addEventListener("mousedown", (e) => {
     if (clickedElement.tagName === 'circle' || clickedElement.tagName === 'polygon') {
         isDragging = true;
         selectedShapeId = clickedElement.id;
-        console.log("selectedShapeId:",selectedShapeId)
-        // Get SVG coordinates
-        // TODO reenable ?
-/*         const pt = container.createSVGPoint();
-        pt.x = e.clientX;
-        pt.y = e.clientY;
-        const svgP = pt.matrixTransform(container.getScreenCTM().inverse());
-        console.log("svgP:",svgP)
-        // Get the shape from the document
-        const shape = handle.docSync().shapes.find(s => s.id === selectedShapeId);
-        if (shape) {
-            offsetX = svgP.x - shape.x;
-            offsetY = svgP.y - shape.y;
-        }
-        console.log("offsetX:",offsetX)
-        console.log("offsetY:",offsetY) */
     }
 });
 
 // Throttled version of moveShape
 const throttledMoveShape = throttle((shapeId, x, y) => {
-  console.log("throttledMoveShape shapeId:",shapeId)
   moveShape(shapeId, x, y);
-  // Pass the moving shape ID to drawAllShapesOnCanvas
-  // TODO no longer needed ?drawAllShapesOnCanvas(container, handle.docSync().shapes)
 }, 16); // 50ms throttle time (20 updates per second)
 
 // Mousemove: Shape verschieben, sofern dragging aktiv ist
@@ -98,9 +75,7 @@ container.addEventListener("mousemove", (e) => {
 
   const newX = svgP.x - offsetX;
   const newY = svgP.y - offsetY;
-console.log("newX:",newX)
-console.log("newY:",newY)
-console.log("selectedShapeId:",selectedShapeId)
+
   throttledMoveShape(selectedShapeId, newX, newY);
 });