From cc3c91b089013e25b0296bfdea6ad9a38687d657 Mon Sep 17 00:00:00 2001
From: Kebechet <samosidor@gmail.com>
Date: Sat, 25 Jul 2026 19:33:08 +0200
Subject: [PATCH 1/2] Make verbose drag logging opt-in instead of always on

---
 BlazorSortableList.Lib/SortableList.razor.js | 28 ++++++++++++--------
 1 file changed, 17 insertions(+), 11 deletions(-)

diff --git a/BlazorSortableList.Lib/SortableList.razor.js b/BlazorSortableList.Lib/SortableList.razor.js
index eb13009..6276e5e 100644
--- a/BlazorSortableList.Lib/SortableList.razor.js
+++ b/BlazorSortableList.Lib/SortableList.razor.js
@@ -1,7 +1,13 @@
+let debugMode = false;
+
+/** Turns on verbose drag/drop tracing. Off by default so consumers get a quiet console. */
+export function setDebugMode(isEnabled) {
+    debugMode = isEnabled;
+}
+
 export function init(id, group, pull, put, sort, handle, filter, component, forceFallback, cssForSelection, multiDragKey, avoidImplicitDeselect, fallbackOnBody, swapThreshold) {
 
-    const DEBUG_MODE = true;
-    if (DEBUG_MODE) {
+    if (debugMode) {
         console.log("Init for Id:", id, "swapThreshold:", swapThreshold);
     }
     let multiDrag = (typeof cssForSelection !== 'undefined');
@@ -17,7 +23,7 @@ export function init(id, group, pull, put, sort, handle, filter, component, forc
         console.error("SortableList init failed: global Sortable not found. Make sure Sortable.js is loaded before this module.", id);
         return;
     }
-    if (DEBUG_MODE && SortableCtor.version) {
+    if (debugMode && SortableCtor.version) {
         console.log("Sortable version:", SortableCtor.version);
     }
 
@@ -70,7 +76,7 @@ export function init(id, group, pull, put, sort, handle, filter, component, forc
         avoidImplicitDeselect: avoidImplicitDeselect,
         swapThreshold: swapThreshold, //0.65,
         onUpdate: (event) => {
-            if (DEBUG_MODE) {
+            if (debugMode) {
                 console.log("onUpdate:");
                 console.log(event);
             }
@@ -96,7 +102,7 @@ export function init(id, group, pull, put, sort, handle, filter, component, forc
                     event.to.insertBefore(item.multiDragElement, event.to.childNodes[item.index]);
                 });
             } else {
-                if (DEBUG_MODE) {
+                if (debugMode) {
                     //console.log("remove item for update:");
                     //console.log(event.item);
                     //console.log("insert it before:", event.to, event.oldIndex, event.to.childNodes, event.to.childNodes[event.oldIndex]);
@@ -111,7 +117,7 @@ export function init(id, group, pull, put, sort, handle, filter, component, forc
             component.invokeMethodAsync('OnUpdateJS', oldIndex, newIndex, event.from.id);
         },
         onRemove: (event) => {
-            if (DEBUG_MODE) {
+            if (debugMode) {
                 console.log("onRemove:");
                 console.log(event);
             }
@@ -150,14 +156,14 @@ export function init(id, group, pull, put, sort, handle, filter, component, forc
             component.invokeMethodAsync('OnRemoveJS', oldIndex, newIndex, event.from.id, event.to.id);
         },
         onSelect: (event) => {
-            if (DEBUG_MODE) {
+            if (debugMode) {
                 console.log("onSelect:");
                 console.log(event);
             }
 
             let children = Array.from(event.from.children);
             let index = children.indexOf(event.item);
-            if (DEBUG_MODE) {
+            if (debugMode) {
                 //console.log(children);
                 console.log(index);
             }
@@ -166,14 +172,14 @@ export function init(id, group, pull, put, sort, handle, filter, component, forc
             component.invokeMethodAsync('OnSelectJS', event.from.id, index);
         },
         onDeselect: (event) => {
-            if (DEBUG_MODE) {
+            if (debugMode) {
                 console.log("onDeselect:");
                 console.log(event);
             }
 
             let children = Array.from(event.to.children);
             let index = children.indexOf(event.item);
-            if (DEBUG_MODE) {
+            if (debugMode) {
                 console.log(index);
             }
 
@@ -181,7 +187,7 @@ export function init(id, group, pull, put, sort, handle, filter, component, forc
             component.invokeMethodAsync('OnDeselectJS', event.from.id, index);
         },
         onEnd:(event) => {
-            if (DEBUG_MODE) {
+            if (debugMode) {
                 console.log("onEnd:");
                 console.log(event);
             }

From cf7ac797004edd33bc233f006cfdea2e5a52f938 Mon Sep 17 00:00:00 2001
From: Kebechet <samosidor@gmail.com>
Date: Sat, 25 Jul 2026 19:39:15 +0200
Subject: [PATCH 2/2] Address review: match setDebugMode strictly instead of by
 truthiness

---
 BlazorSortableList.Lib/SortableList.razor.js | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/BlazorSortableList.Lib/SortableList.razor.js b/BlazorSortableList.Lib/SortableList.razor.js
index 6276e5e..828457d 100644
--- a/BlazorSortableList.Lib/SortableList.razor.js
+++ b/BlazorSortableList.Lib/SortableList.razor.js
@@ -1,8 +1,11 @@
 let debugMode = false;
 
-/** Turns on verbose drag/drop tracing. Off by default so consumers get a quiet console. */
+/**
+ * Turns on verbose drag/drop tracing. Off by default so consumers get a quiet console.
+ * Matched strictly rather than by truthiness, so the string "false" does not switch logging on.
+ */
 export function setDebugMode(isEnabled) {
-    debugMode = isEnabled;
+    debugMode = isEnabled === true || isEnabled === 'true';
 }
 
 export function init(id, group, pull, put, sort, handle, filter, component, forceFallback, cssForSelection, multiDragKey, avoidImplicitDeselect, fallbackOnBody, swapThreshold) {