Couple more comments

Author: UnityGuillaume

Assets/Scripts/Gameplay/EntryPoint.cs

@@ -24,17 +24,17 @@ public static EntryPoint Instance
     }
 
     /// <summary>
-    /// This be called either
-    /// - By the Menu script that will create it at the game launch (in a "real" game, there could be a Loading scene loaded
+    /// This is called either by :
+    /// - the Menu script that will create it at the game launch (in a full game, there could be a Loading scene loaded
     /// in first that will create it and display a progress bar possibly)
     /// - On the first call to Instance if it wasn't created (in editor only, see comment in that function)
     ///
     /// It take care of creating all globals data and initialize all system that need to be (like databases).
     /// </summary>
     public static void Create()
     {
-        //in this sample we use the main menu scene as initialisation. That mean going back to main menu can retrigger
-        //This initialization, so we check if we were already initialized. In a full project, this init would be called
+        //in this sample we use the main menu scene as initialisation. That mean going back to main menu can trigger again
+        //this initialization. So we check if we were already initialized. In a full project, this Create would be called
         //in a scene only run when the game launch, *before* main menu scene.
         if(s_Instance != null)
             return;
@@ -48,6 +48,7 @@ public static void Create()
 
         DontDestroyOnLoad(s_Instance);
 
+        //This will init the Database and create the lookup dictionary matching an Item ScriptableObject and it's unique ID
         s_Instance.ItemDB.Init();
 
         s_Instance.m_PanelInstance = Instantiate(s_Instance.PanelPrefab);

Assets/Scripts/ItemDatabase.cs

@@ -9,7 +9,8 @@
 /// <summary>
 /// This keep a list of all items in the game, and build a dictionary matching their UniqueId to their instance when the Init
 /// function is called (called by the EntryPoint initialization). This allow to have a single entry point to looking up
-/// all object in the game (e.g. save file only contain the uniqueId of the item, that is looked up on load in this) 
+/// all object in the game (e.g. save file only contain the uniqueId of the item, that is looked up on load in this).
+/// The Items list is built in the Editor on the ScriptableObject instance.
 /// </summary>
 [CreateAssetMenu(fileName = "Itemdb.asset", menuName = "SaveLoad/Item Database")]
 public class ItemDatabase : ScriptableObject

Assets/Scripts/UI/ConfirmationPopup.cs

@@ -2,6 +2,10 @@
 using System.Collections.Generic;
 using UnityEngine;
 
+/// <summary>
+/// A simple popup that display a confirm/cancel button combo and with user defined callback.
+/// Used in the SaveGameGrid to confirm overwriting a save file.
+/// </summary>
 public class ConfirmationPopup : MonoBehaviour
 {
     private System.Action<bool> m_OnClose;

Assets/Scripts/UI/LoadingPanel.cs

@@ -6,7 +6,7 @@
 
 /// <summary>
 /// Quick helper that will fade a black screen in and out, with the possibility to give a function to call when the fade
-/// in finish (so the screen is full dark)
+/// is halfway through (so the screen is full dark)
 /// </summary>
 public class LoadingPanel : MonoBehaviour
 {

Assets/Scripts/UI/UIHandler.cs

@@ -4,6 +4,9 @@
 using UnityEngine;
 using UnityEngine.UI;
 
+/// <summary>
+/// Allow for an access from anywhere to the UI that display the Inventory to easily update it.
+/// </summary>
 public class UIHandler : MonoBehaviour
 {
     public static UIHandler Instance => s_Instance;