Merge pull request #5644 from subhraneel2005/master

chore: remove debug console statements from js/palette.js

Author: omsuneri

js/palette.js

@@ -104,9 +104,6 @@ class Palettes {
     }
 
     _makeSelectorButton(i) {
-        // eslint-disable-next-line no-console
-        console.debug("makeSelectorButton " + i);
-
         if (!document.getElementById("palette")) {
             const element = document.createElement("div");
             element.id = "palette";
@@ -134,6 +131,7 @@ class Palettes {
             element.childNodes[0].style.border = `1px solid ${platformColor.selectorSelected}`;
             document.body.appendChild(element);
         }
+
         const tr = docById("palette").children[0].children[0].children[0].children[0];
         const td = tr.insertCell();
         td.width = 1.5 * this.cellSize;
@@ -218,8 +216,6 @@ class Palettes {
     }
 
     getPluginMacroExpansion(blkname, x, y) {
-        // eslint-disable-next-line no-console
-        console.debug(this.pluginMacros[blkname]);
         const obj = this.pluginMacros[blkname];
         if (obj != null) {
             obj[0][2] = x;
@@ -638,9 +634,9 @@ class PaletteModel {
         }
 
         const protoBlock = this.activity.blocks.protoBlockDict[blkname];
+
         if (protoBlock === null) {
-            // eslint-disable-next-line no-console
-            console.debug("Could not find block " + blkname);
+            return;
         }
 
         let label = "";
@@ -1170,8 +1166,6 @@ class Palette {
 
     _makeBlockFromPalette(protoblk, blkname, callback) {
         if (protoblk === null) {
-            // eslint-disable-next-line no-console
-            console.debug("null protoblk?");
             return;
         }
 
@@ -1191,10 +1185,7 @@ class Palette {
                 break;
             case "storein2":
                 // Use the name of the box in the label
-                // eslint-disable-next-line no-console
-                console.debug(
-                    "storein2" + " " + protoblk.defaults[0] + " " + protoblk.staticLabels[0]
-                );
+
                 blkname = "store in2 " + protoblk.defaults[0];
                 newBlk = protoblk.name;
                 arg = protoblk.staticLabels[0];
@@ -1211,8 +1202,6 @@ class Palette {
                     blkname = "namedbox";
                     arg = _("box");
                 } else {
-                    // eslint-disable-next-line no-console
-                    console.debug(protoblk.defaults[0]);
                     blkname = protoblk.defaults[0];
                     arg = protoblk.defaults[0];
                 }
@@ -1348,9 +1337,6 @@ class Palette {
                 for (let blk = 0; blk < this.activity.blocks.blockList.length; blk++) {
                     const block = this.activity.blocks.blockList[blk];
                     if (block.name === "status" && !block.trash) {
-                        console.log(
-                            "Status block already exists, preventing creation of another one"
-                        );
                         return;
                     }
                 }
@@ -1474,7 +1460,6 @@ class Palette {
                 for (let i = 0; i < boxBlocks.length; i++) {
                     const boxBlockId = boxBlocks[i];
                     const boxBlock = activity.blocks.blockList[boxBlockId];
-                    console.log("Adding box block to status:", boxBlock);
 
                     statusBlocks.push([
                         lastBlockIndex + 1,
@@ -1499,7 +1484,7 @@ class Palette {
                     ]);
                     lastBlockIndex += 2;
                 }
-                console.log("blocks");
+
                 macroExpansion = statusBlocks;
 
                 // Initialize the status matrix
@@ -1637,8 +1622,7 @@ const initPalettes = async palettes => {
 
     palettes.init_selectors();
     palettes.makePalettes(0);
-    // eslint-disable-next-line no-console
-    console.debug("Time to show the palettes.");
+
     palettes.show();
 };