From 5244bbcfbb7adc1f570051b10f063e855c3e22a6 Mon Sep 17 00:00:00 2001
From: Felix Boucher <felix-gabriel.boucher-luneau.1@ens.etsmtl.ca>
Date: Tue, 13 Jun 2023 18:35:08 -0400
Subject: [PATCH] change names + add commentaries

---
 .../Drag&Drop/GameObjectPlacementButton.cs    | 19 ++++++++++---------
 .../Scripts/Drag&Drop/TilePlacementButton.cs  |  8 ++++----
 2 files changed, 14 insertions(+), 13 deletions(-)

diff --git a/Assets/Scripts/Drag&Drop/GameObjectPlacementButton.cs b/Assets/Scripts/Drag&Drop/GameObjectPlacementButton.cs
index f51d073..a4c1d7c 100644
--- a/Assets/Scripts/Drag&Drop/GameObjectPlacementButton.cs
+++ b/Assets/Scripts/Drag&Drop/GameObjectPlacementButton.cs
@@ -8,23 +8,24 @@ public class GameObjectPlacementButton : UnitPlacementButton
     {
         var instance = Instantiate(_prefab);
 
-        //we need to do that before stripping the object
+        //we need to fetch the detection size before stripping the object
         var detectComp = instance.GetComponentInChildren<Detection>();
         Rect detectionRect = default;
         if(detectComp) detectionRect = instance.GetComponentInChildren<Detection>().DetectionRectangle;
 
-        //strip the objecty
-        foreach (var comp in instance.transform.GetAllComponents<Rigidbody2D>()) Destroy(comp);
-        foreach (var comp in instance.transform.GetAllComponents<Collider2D>()) Destroy(comp);
-        foreach (var comp in instance.transform.GetAllComponents<MonoBehaviour>()) Destroy(comp);
+        //strip the object
+        foreach (var r_body in instance.transform.GetAllComponents<Rigidbody2D>()) Destroy(r_body);
+        foreach (var coll in instance.transform.GetAllComponents<Collider2D>()) Destroy(coll);
+        foreach (var script in instance.transform.GetAllComponents<MonoBehaviour>()) Destroy(script);
 
-        foreach (var comp in instance.transform.GetAllComponents<SpriteRenderer>())
+        //assign outline material tou all renderers of the placeholder
+        foreach (var rend in instance.transform.GetAllComponents<SpriteRenderer>())
         {
-            var color = (comp as SpriteRenderer).color;
+            var color = (rend as SpriteRenderer).color;
             color.a = 0.6f;
-            (comp as SpriteRenderer).color = color;
+            (rend as SpriteRenderer).color = color;
 
-            (comp as SpriteRenderer).material = _outlineMaterial;
+            (rend as SpriteRenderer).material = _outlineMaterial;
         }
         
 
diff --git a/Assets/Scripts/Drag&Drop/TilePlacementButton.cs b/Assets/Scripts/Drag&Drop/TilePlacementButton.cs
index d1181a9..5015048 100644
--- a/Assets/Scripts/Drag&Drop/TilePlacementButton.cs
+++ b/Assets/Scripts/Drag&Drop/TilePlacementButton.cs
@@ -7,10 +7,10 @@ public class TilePlacementButton : UnitPlacementButton
     public LevelTile tile;
     protected override void Place()
     {
-        var placeholder = new GameObject(tile.name);
-        var comp = placeholder.AddComponent<TilePlaceholder>();
-        comp.Tile = tile;
-        var rend = placeholder.AddComponent<SpriteRenderer>();
+        var instance = new GameObject(tile.name);
+        var placeholder = instance.AddComponent<TilePlaceholder>();
+        placeholder.Tile = tile;
+        var rend = instance.AddComponent<SpriteRenderer>();
         rend.sprite = tile.Sprite;
         rend.sortingLayerName = "Character";
         rend.material = _outlineMaterial;