From e784b27848d9a8067c2e8f8daaa99174f8f2b3eb Mon Sep 17 00:00:00 2001
From: Karl Pannek <logspum@gmail.com>
Date: Fri, 18 Jul 2025 23:05:55 +0200
Subject: [PATCH] Fix debug draw overlay visibility - simple canvas display
 toggle

- Fixed debug draw overlay not disappearing when checkbox unchecked
- Added canvas.style.display control in onToggleDebugMode
- Kept existing canvas-based PlanckDebugDraw system
- Both canvas overlay and PIXI debug layer are now properly toggled
- Debug physics bodies now properly show/hide with checkbox
---
 app/Game/Client/Physics/Engine.js            |  5 +++++
 app/Game/Client/View/Pixi/PlanckDebugDraw.js | 18 +++++-------------
 2 files changed, 10 insertions(+), 13 deletions(-)

diff --git a/app/Game/Client/Physics/Engine.js b/app/Game/Client/Physics/Engine.js
index bca83a5..1f6be39 100755
--- a/app/Game/Client/Physics/Engine.js
+++ b/app/Game/Client/Physics/Engine.js
@@ -29,6 +29,11 @@ function (Parent, Settings, domController, Box2D, nc, PlanckDebugDraw, debugLaye
             this.setupDebugDraw();
         }
 
+        // Show/hide the debug canvas overlay
+        if (this.debugCanvas) {
+            this.debugCanvas.style.display = this.debugMode ? 'block' : 'none';
+        }
+
         debugLayer.container.visible = this.debugMode;
     };
 
diff --git a/app/Game/Client/View/Pixi/PlanckDebugDraw.js b/app/Game/Client/View/Pixi/PlanckDebugDraw.js
index c294b74..e0a5f6e 100644
--- a/app/Game/Client/View/Pixi/PlanckDebugDraw.js
+++ b/app/Game/Client/View/Pixi/PlanckDebugDraw.js
@@ -59,11 +59,6 @@ function (Settings) {
                     continue;
                 }
                 
-                // Skip non-sensor fixtures if they were sensors in the old logic
-                if (!isSensor && !Settings.DEBUG_DRAW_SENSORS) {
-                    // This was the old logic - skip sensors, but we're keeping non-sensors
-                }
-                
                 // Set colors based on sensor status and body type
                 if (isSensor) {
                     // Sensors: orange with more visibility, no stroke
@@ -92,7 +87,6 @@ function (Settings) {
             for (var body = world.getBodyList(); body; body = body.getNext()) {
                 var transform = body.getTransform();
                 this.drawCenterOfMass(transform);
-                // Removed drawBodyOrigin call since origin and center of mass are the same in Planck.js
             }
         }
 
@@ -163,17 +157,15 @@ function (Settings) {
         }
     };
 
-    PlanckDebugDraw.prototype.drawEdge = function(shape, transform, isSensor) {
+    PlanckDebugDraw.prototype.drawEdge = function(shape, transform, isSensor, fillColor, lineColor, lineWidth) {
         var v1 = this.transformVertex(shape.m_vertex1, transform);
         var v2 = this.transformVertex(shape.m_vertex2, transform);
         
-        this.ctx.beginPath();
-        this.ctx.moveTo(v1.x, v1.y);
-        this.ctx.lineTo(v2.x, v2.y);
-        
         // Only draw stroke for non-sensors
-        if (!isSensor) {
-            this.ctx.stroke();
+        if (!isSensor && lineWidth > 0) {
+            this.graphics.lineStyle(lineWidth, lineColor);
+            this.graphics.moveTo(v1.x * this.scale, v1.y * this.scale);
+            this.graphics.lineTo(v2.x * this.scale, v2.y * this.scale);
         }
     };