fixed bug with missclicks in the background

Assets/Scripts/Global Scripts/QuestionManager.cs

@@ -4,14 +4,15 @@
 using TMPro;
 using Unity.VisualScripting;
 using UnityEngine;
+using UnityEngine.EventSystems;
 using UnityEngine.SceneManagement;
 using UnityEngine.UI;
 using UnityEngine.UIElements;
 
 /// <summary>
 /// Handles how when questions appear and evaluates the player's answer
 /// </summary>
-public class QuestionManager : MonoBehaviour
+public class QuestionManager : MonoBehaviour, IPointerEnterHandler, IPointerExitHandler
 {
     [Header("Attributes")]
     private List<QuestionData> questions;
@@ -142,6 +143,8 @@ public bool CheckAnswer()
     public void CloseQuestionUI()
     {
         UIManager.Instance.SetHoveringState(false);
+        rightAnswer.SetActive(false);
+        wrongAnswer.SetActive(false);
         ActivateCanvas(false);
     }
 
@@ -162,4 +165,22 @@ private void Quit()
         SceneManager.UnloadSceneAsync("Question");
         UIManager.Instance.SetHoveringState(false);
     }
+
+    /// <summary>
+    ///  This function sets the setHoveringState function to true if the mouse is over the menu
+    /// </summary>
+    /// <param name="eventData"> The mouse</param>
+    public void OnPointerEnter(PointerEventData eventData)
+    {
+        UIManager.Instance.SetHoveringState(true);
+    }
+
+    /// <summary>
+    ///  This function sets the setHoveringState function to false if the mouse is over the menu
+    /// </summary>
+    /// <param name="eventData"> The mouse</param>
+    public void OnPointerExit(PointerEventData eventData)
+    {
+        UIManager.Instance.SetHoveringState(false);
+    }
 }

Assets/Scripts/HUD/PauseButton.cs

@@ -97,6 +97,5 @@ public void OnPointerExit(PointerEventData eventData)
     {
         mouseOver = false;
         UIManager.Instance.SetHoveringState(false);
-
     }
 }

Assets/Scripts/HUD/PauseMenu.cs

@@ -2,14 +2,15 @@
 using System.Collections.Generic;
 using Unity.VisualScripting;
 using UnityEngine;
+using UnityEngine.UI;
 using UnityEngine.EventSystems;
 using UnityEngine.SceneManagement;
 using TMPro;
 
 /// <summary>
 ///  This class manages the pause menu including its resume and pause logic
 /// </summary>
-public class PauseMenu : MonoBehaviour
+public class PauseMenu : MonoBehaviour, IPointerEnterHandler, IPointerExitHandler
 {
     private bool mouseOver = false;
 
@@ -20,6 +21,7 @@ public void Resume()
     {
         Time.timeScale = 1f;
         SceneManager.UnloadSceneAsync("Pause");
+        UIManager.Instance.SetHoveringState(false);
     }
 
     /// <summary>

Assets/Scripts/Map/Plots.cs

@@ -29,7 +29,7 @@ private void Start()
     /// </summary>
     private void OnMouseEnter()
     {
-        if (tileSprite != null)
+        if (tileSprite != null && BuildManager.Instance.GetSelectedTowerIndex() != -1)
         {
             tileSprite.color = hoverColor;
         }
@@ -40,7 +40,7 @@ private void OnMouseEnter()
     /// </summary>
     private void OnMouseExit()
     {
-        if (tileSprite != null)
+        if (tileSprite != null && BuildManager.Instance.GetSelectedTowerIndex() != -1)
         {
             tileSprite.color = startColor;
         }
@@ -59,7 +59,7 @@ private void OnMouseDown()
 
         if (UIManager.Instance.IsHoveringUI()) return;
 
-        if (towerObject != null)
+        if (towerObject != null&& BuildManager.Instance.GetSelectedTowerIndex() != -1)
         {
             tower.OpenUpgradeUI();
             return;

Assets/Scripts/Tower/BuildManager.cs

@@ -11,7 +11,7 @@ public class BuildManager : MonoBehaviour
     [Header("References")]
     [SerializeField] private Tower[] towers;
 
-    private int selectedTower = 0;
+    private int selectedTower = -1;
 
     /// <summary>
     /// This function manages the singleton instance, so it initializes the <c>instance</c> variable, if not set, or
@@ -47,4 +47,9 @@ public void SetSelectedTower(int selectedTower)
     {
         this.selectedTower = selectedTower;
     }
+
+    public int GetSelectedTowerIndex()
+    {
+        return selectedTower;
+    }
 }

Assets/Scripts/Tower/TowerTypes/BaseTower.cs

@@ -77,7 +77,6 @@ public void OpenUpgradeUI()
     public void CloseUpgradeUI()
     {
         upgradeUI.SetActive(false);
-        UIManager.Instance.SetHoveringState(false);
     }
 
     /// <summary>
@@ -104,7 +103,8 @@ public void Answer(bool answer)
         else
         {
             Debug.Log("wrong answer");
-            // punishmenet: player looses half the price of the upgrade in credits
+
+            // TODO: adjust punishment
             LevelManager.Instance.SpendCurrency(CalculateCost() / 2);
         }
 
@@ -155,6 +155,5 @@ private int CalculateCost()
     {
         return Mathf.RoundToInt(baseUpgradeCost * Mathf.Pow(level, CostExponent));
     }
-
-
+
 }

Assets/Scripts/UIManager.cs

@@ -8,7 +8,7 @@
 public class UIManager : MonoBehaviour
 {
     public static UIManager Instance { get; private set; }
-    private bool isHoveringUI;
+    private bool isHoveringUI = true;
 
     /// <summary>
     /// This function manages the singleton instance, so it initializes the <c>instance</c> variable, if not set, or
@@ -30,7 +30,7 @@ private void Awake()
     /// <summary>
     ///  This function detects whether the mouse is currently hovering over an UI element
     /// </summary>
-    /// <param name="state"> The state of where the mouse is at the moment</param>
+    /// <param name="state">The state whether the player hovers over an UI element</param>
     public void SetHoveringState(bool state)
     {
         isHoveringUI = state;