From 2994d6185019cc5af93f738a985e9f1580d31519 Mon Sep 17 00:00:00 2001
From: JoshuaMoelans <60878493+JoshuaMoelans@users.noreply.github.com>
Date: Fri, 30 Jan 2026 11:53:27 +0100
Subject: [PATCH 1/7] add controller UI navigation/submit - also auto-selects
 one upgrade, and can now confirm with spacebar on keyboard too

---
 Assets/Scripts/Upgrades/LevelUpUI.cs          | 29 ++++++++++---------
 .../Settings/InputSystem_Actions.inputactions | 11 +++++++
 2 files changed, 26 insertions(+), 14 deletions(-)

diff --git a/Assets/Scripts/Upgrades/LevelUpUI.cs b/Assets/Scripts/Upgrades/LevelUpUI.cs
index a65c373..6cfa394 100644
--- a/Assets/Scripts/Upgrades/LevelUpUI.cs
+++ b/Assets/Scripts/Upgrades/LevelUpUI.cs
@@ -49,6 +49,10 @@ private void OnEnable()
             InputSystem.actions.FindActionMap("Player").Disable();
             InputSystem.actions.FindActionMap("UI").Enable();
             
+            // Subscribe to input events
+            _navigateAction.performed += OnNavigatePerformed;
+            _submitAction.performed += OnSubmitPerformed;
+            
             // Pause the game
             Time.timeScale = 0;
 
@@ -75,6 +79,9 @@ private void OnEnable()
 
             _option1Button.onClick.AddListener(() => SelectUpgrade(upgradeChoice1));
             _option2Button.onClick.AddListener(() => SelectUpgrade(upgradeChoice2));
+            
+            // Set initial highlighted button to option 1
+            SetHighlightedButton(_option1Button);
 
             if (_demoConfig != null && _demoConfig.AutoPlay)
             {
@@ -107,19 +114,14 @@ private IEnumerator SelectSomething()
             _highlightedButton?.GetComponent<Button>().onClick.Invoke();
         }
         
-        public void OnNavigate()
+        private void OnNavigatePerformed(InputAction.CallbackContext context)
         {
             if (!gameObject.activeSelf)
             {
                 return;
             }
             
-            if (!_navigateAction.WasPressedThisFrame())
-            {
-                return;
-            }
-            
-            var direction = _navigateAction.ReadValue<Vector2>();
+            var direction = context.ReadValue<Vector2>();
             if (direction.x < 0)
             {
                 SetHighlightedButton(_option1Button);
@@ -130,19 +132,14 @@ public void OnNavigate()
             }
         }
         
-        public void OnSubmit()
+        private void OnSubmitPerformed(InputAction.CallbackContext context)
         {
             if (!gameObject.activeSelf)
             {
                 return;
             }
             
-            if (!_submitAction.IsPressed())
-            {
-                return;
-            }
-            
-            _highlightedButton?.GetComponent<Button>().onClick.Invoke();
+            _highlightedButton?.onClick.Invoke();
         }
 
         public void SetHighlightedButton(Button button)
@@ -156,6 +153,10 @@ public void SetHighlightedButton(Button button)
 
         private void OnDisable()
         {
+            // Unsubscribe from input events
+            _navigateAction.performed -= OnNavigatePerformed;
+            _submitAction.performed -= OnSubmitPerformed;
+            
             _option1Button.onClick.RemoveAllListeners();
             _option2Button.onClick.RemoveAllListeners();
         }
diff --git a/Assets/Settings/InputSystem_Actions.inputactions b/Assets/Settings/InputSystem_Actions.inputactions
index 6a1dd7b..5a29f87 100644
--- a/Assets/Settings/InputSystem_Actions.inputactions
+++ b/Assets/Settings/InputSystem_Actions.inputactions
@@ -764,6 +764,17 @@
                     "isComposite": false,
                     "isPartOfComposite": false
                 },
+                {
+                    "name": "",
+                    "id": "a1b2c3d4-5e6f-7a8b-9c0d-1e2f3a4b5c6d",
+                    "path": "<Keyboard>/space",
+                    "interactions": "",
+                    "processors": "",
+                    "groups": "Keyboard&Mouse",
+                    "action": "Submit",
+                    "isComposite": false,
+                    "isPartOfComposite": false
+                },
                 {
                     "name": "",
                     "id": "82627dcc-3b13-4ba9-841d-e4b746d6553e",

From 4dc37629dfbdf56d948df87ab605a464a969d14d Mon Sep 17 00:00:00 2001
From: JoshuaMoelans <60878493+JoshuaMoelans@users.noreply.github.com>
Date: Fri, 30 Jan 2026 12:01:55 +0100
Subject: [PATCH 2/7] fix double pause menu

---
 .../SceneManagers/BattleSceneManager.cs       |  6 +++++
 .../Settings/InputSystem_Actions.inputactions | 22 +++++++++++++++++++
 2 files changed, 28 insertions(+)

diff --git a/Assets/Scripts/SceneManagers/BattleSceneManager.cs b/Assets/Scripts/SceneManagers/BattleSceneManager.cs
index 15b6628..5d3dbbf 100644
--- a/Assets/Scripts/SceneManagers/BattleSceneManager.cs
+++ b/Assets/Scripts/SceneManagers/BattleSceneManager.cs
@@ -463,6 +463,12 @@ private void SetCurrentLevel(int level)
 
     public void OnPause()
     {
+        // Don't allow pausing if the level up UI is active (it already pauses the game)
+        if (_levelUpUI.activeSelf)
+        {
+            return;
+        }
+        
         if (_gameState == GameState.Playing)
         {
             PauseGame();
diff --git a/Assets/Settings/InputSystem_Actions.inputactions b/Assets/Settings/InputSystem_Actions.inputactions
index 5a29f87..e146bdc 100644
--- a/Assets/Settings/InputSystem_Actions.inputactions
+++ b/Assets/Settings/InputSystem_Actions.inputactions
@@ -239,6 +239,17 @@
                     "isComposite": false,
                     "isPartOfComposite": false
                 },
+                {
+                    "name": "",
+                    "id": "d1e2f3a4-b5c6-7d8e-9f0a-1b2c3d4e5f6a",
+                    "path": "<DualShockGamepad>/start",
+                    "interactions": "",
+                    "processors": "",
+                    "groups": ";Gamepad",
+                    "action": "Pause",
+                    "isComposite": false,
+                    "isPartOfComposite": false
+                },
                 {
                     "name": "",
                     "id": "0328a504-56c9-413d-b17e-f54c8515dee2",
@@ -753,6 +764,17 @@
                     "isComposite": false,
                     "isPartOfComposite": false
                 },
+                {
+                    "name": "",
+                    "id": "e1f2a3b4-c5d6-7e8f-9a0b-1c2d3e4f5a6b",
+                    "path": "<DualShockGamepad>/start",
+                    "interactions": "",
+                    "processors": "",
+                    "groups": ";Gamepad",
+                    "action": "Pause",
+                    "isComposite": false,
+                    "isPartOfComposite": false
+                },
                 {
                     "name": "",
                     "id": "9e92bb26-7e3b-4ec4-b06b-3c8f8e498ddc",

From 851e6b16590921e214b574a6126453252281ba33 Mon Sep 17 00:00:00 2001
From: JoshuaMoelans <60878493+JoshuaMoelans@users.noreply.github.com>
Date: Fri, 30 Jan 2026 12:07:43 +0100
Subject: [PATCH 3/7] pause UI navigation

---
 Assets/Scripts/SceneManagers/HUDManager.cs | 19 +++++++++++--------
 Assets/Scripts/UI/HUD/HUD.cs               |  2 ++
 2 files changed, 13 insertions(+), 8 deletions(-)

diff --git a/Assets/Scripts/SceneManagers/HUDManager.cs b/Assets/Scripts/SceneManagers/HUDManager.cs
index dfb01ae..85a2eec 100644
--- a/Assets/Scripts/SceneManagers/HUDManager.cs
+++ b/Assets/Scripts/SceneManagers/HUDManager.cs
@@ -23,12 +23,20 @@ private void Awake()
             _navigateAction = InputSystem.actions.FindAction("Navigate");
             _submitAction = InputSystem.actions.FindAction("Submit");
             
-            // Subscribe to the performed callback only
+            // Subscribe to input events
+            _navigateAction.performed += OnNavigatePerformed;
             _submitAction.performed += OnSubmitPerformed;
             
             _tryAgainHighlighter = tryAgainButton.GetComponent<Highlighter>();
             _quitHighlighter = quitButton.GetComponent<Highlighter>();
         }
+        
+        private void OnDestroy()
+        {
+            // Unsubscribe from input events
+            _navigateAction.performed -= OnNavigatePerformed;
+            _submitAction.performed -= OnSubmitPerformed;
+        }
 
         private void OnSubmitPerformed(InputAction.CallbackContext context)
         {
@@ -40,7 +48,7 @@ private void OnSubmitPerformed(InputAction.CallbackContext context)
             _highlightedButton?.GetComponent<Button>().onClick.Invoke();
         }
 
-        public void OnNavigate()
+        private void OnNavigatePerformed(InputAction.CallbackContext context)
         {
             if (!gameObject.activeSelf)
             {
@@ -52,12 +60,7 @@ public void OnNavigate()
                 return;
             }
 
-            if (!_navigateAction.WasPressedThisFrame())
-            {
-                return;
-            }
-
-            var direction = _navigateAction.ReadValue<Vector2>();
+            var direction = context.ReadValue<Vector2>();
             
             // Simple navigation between try again and quit buttons
             if (_highlightedButton == null)
diff --git a/Assets/Scripts/UI/HUD/HUD.cs b/Assets/Scripts/UI/HUD/HUD.cs
index 4daca78..5b8d293 100644
--- a/Assets/Scripts/UI/HUD/HUD.cs
+++ b/Assets/Scripts/UI/HUD/HUD.cs
@@ -1,4 +1,5 @@
 using System.Runtime.InteropServices;
+using SceneManagers;
 using TMPro;
 using UnityEngine;
 using UnityEngine.UI;
@@ -16,6 +17,7 @@ public class HUD : MonoBehaviour
     [SerializeField] private ScorePoster _scorePoster;
     [SerializeField] private GameObject _tryAgain;
     [SerializeField] private GameObject _quit;
+    [SerializeField] private HUDManager _hudManager;
 
     private DemoConfiguration _demoConfig;
     private XpBar _xpBar;

From e42556873b411243b4f5cd928bd27855ff4b1d92 Mon Sep 17 00:00:00 2001
From: JoshuaMoelans <60878493+JoshuaMoelans@users.noreply.github.com>
Date: Fri, 30 Jan 2026 12:15:59 +0100
Subject: [PATCH 4/7] show directional arrow for gamepad

---
 Assets/Scripts/Characters/Arrow.cs | 15 +++++++++------
 1 file changed, 9 insertions(+), 6 deletions(-)

diff --git a/Assets/Scripts/Characters/Arrow.cs b/Assets/Scripts/Characters/Arrow.cs
index ecae455..8c58edc 100644
--- a/Assets/Scripts/Characters/Arrow.cs
+++ b/Assets/Scripts/Characters/Arrow.cs
@@ -1,19 +1,22 @@
 using UnityEngine;
+using UnityEngine.InputSystem;
 
 public class Arrow : MonoBehaviour
 {
     [SerializeField] private bool _forceEnable;
+    
     private void Awake()
     {
         if (_forceEnable)
         {
             return;
         }
-            
-        if (Application.platform != RuntimePlatform.Android && 
-            Application.platform != RuntimePlatform.IPhonePlayer)
-        {
-            gameObject.SetActive(false);    
-        }
+        
+        // Show arrow on mobile platforms or when a gamepad is connected
+        bool shouldShowArrow = Application.platform == RuntimePlatform.Android || 
+                               Application.platform == RuntimePlatform.IPhonePlayer ||
+                               Gamepad.current != null;
+        
+        gameObject.SetActive(shouldShowArrow);
     }
 }

From f2df7a6056b82b9a80c73d573861c3b7f7cc0092 Mon Sep 17 00:00:00 2001
From: JoshuaMoelans <60878493+JoshuaMoelans@users.noreply.github.com>
Date: Fri, 30 Jan 2026 12:23:47 +0100
Subject: [PATCH 5/7] prevent UI bleed

---
 Assets/Scripts/SceneManagers/HUDManager.cs | 13 ++++++++++++-
 Assets/Scripts/UI/HUD/HUD.cs               |  6 ++++++
 2 files changed, 18 insertions(+), 1 deletion(-)

diff --git a/Assets/Scripts/SceneManagers/HUDManager.cs b/Assets/Scripts/SceneManagers/HUDManager.cs
index 85a2eec..f8f09cf 100644
--- a/Assets/Scripts/SceneManagers/HUDManager.cs
+++ b/Assets/Scripts/SceneManagers/HUDManager.cs
@@ -45,7 +45,11 @@ private void OnSubmitPerformed(InputAction.CallbackContext context)
                 return;
             }
             
-            _highlightedButton?.GetComponent<Button>().onClick.Invoke();
+            // Only invoke if the highlighted button is actually active
+            if (_highlightedButton != null && _highlightedButton.activeSelf)
+            {
+                _highlightedButton.GetComponent<Button>().onClick.Invoke();
+            }
         }
 
         private void OnNavigatePerformed(InputAction.CallbackContext context)
@@ -95,5 +99,12 @@ public void SetHighlightedButton(Highlighter highlighted)
             highlighted.Highlight();
             _highlightedButton = highlighted.gameObject;
         }
+        
+        public void ClearHighlightedButton()
+        {
+            _tryAgainHighlighter.Highlight(false);
+            _quitHighlighter.Highlight(false);
+            _highlightedButton = null;
+        }
     }
 }
diff --git a/Assets/Scripts/UI/HUD/HUD.cs b/Assets/Scripts/UI/HUD/HUD.cs
index 5b8d293..cf96e7c 100644
--- a/Assets/Scripts/UI/HUD/HUD.cs
+++ b/Assets/Scripts/UI/HUD/HUD.cs
@@ -80,6 +80,12 @@ public void HidePause()
         _gameOverText.enabled = false;
         _tryAgain.SetActive(false);
         _quit.SetActive(false);
+        
+        // Clear the highlighted button to prevent accidental clicks
+        if (_hudManager != null)
+        {
+            _hudManager.ClearHighlightedButton();
+        }
     }
 
     public void ShowGameOver()

From cf33ec9a338c05414480f2a58ca793f0d09abb21 Mon Sep 17 00:00:00 2001
From: JoshuaMoelans <60878493+JoshuaMoelans@users.noreply.github.com>
Date: Tue, 3 Feb 2026 17:13:52 +0100
Subject: [PATCH 6/7] update xbox controller pause button

---
 Assets/Settings/InputSystem_Actions.inputactions | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/Assets/Settings/InputSystem_Actions.inputactions b/Assets/Settings/InputSystem_Actions.inputactions
index e146bdc..fabfbdd 100644
--- a/Assets/Settings/InputSystem_Actions.inputactions
+++ b/Assets/Settings/InputSystem_Actions.inputactions
@@ -209,7 +209,7 @@
                 {
                     "name": "",
                     "id": "516ec204-3b9e-43d6-b1fe-a3b649c01b88",
-                    "path": "<GXDKGamepad>/select",
+                    "path": "<GXDKGamepad>/start",
                     "interactions": "",
                     "processors": "",
                     "groups": ";Gamepad",
@@ -231,7 +231,7 @@
                 {
                     "name": "",
                     "id": "a57c8beb-ff4b-48af-b12b-39a9b69277f8",
-                    "path": "<XInputController>/select",
+                    "path": "<XInputController>/start",
                     "interactions": "",
                     "processors": "",
                     "groups": ";Gamepad",
@@ -734,7 +734,7 @@
                 {
                     "name": "",
                     "id": "4f2f6db6-f487-4f7b-819c-50fda21f90d7",
-                    "path": "<GXDKGamepad>/select",
+                    "path": "<GXDKGamepad>/start",
                     "interactions": "",
                     "processors": "",
                     "groups": ";Gamepad",
@@ -756,7 +756,7 @@
                 {
                     "name": "",
                     "id": "2b76180b-bc1b-4768-ae01-060e98b1d64a",
-                    "path": "<XInputController>/select",
+                    "path": "<XInputController>/start",
                     "interactions": "",
                     "processors": "",
                     "groups": ";Gamepad",

From 25bf3a7c7859b40c53a7ccf4ef587e0995ae063a Mon Sep 17 00:00:00 2001
From: JoshuaMoelans <60878493+JoshuaMoelans@users.noreply.github.com>
Date: Tue, 3 Feb 2026 17:42:41 +0100
Subject: [PATCH 7/7] pause for switch controller

---
 .../Settings/InputSystem_Actions.inputactions | 22 +++++++++++++++++++
 1 file changed, 22 insertions(+)

diff --git a/Assets/Settings/InputSystem_Actions.inputactions b/Assets/Settings/InputSystem_Actions.inputactions
index fabfbdd..3c17883 100644
--- a/Assets/Settings/InputSystem_Actions.inputactions
+++ b/Assets/Settings/InputSystem_Actions.inputactions
@@ -250,6 +250,17 @@
                     "isComposite": false,
                     "isPartOfComposite": false
                 },
+                {
+                    "name": "",
+                    "id": "a1b2c3d4-e5f6-7a8b-9c0d-1e2f3a4b5c6e",
+                    "path": "<SwitchProControllerHID>/start",
+                    "interactions": "",
+                    "processors": "",
+                    "groups": ";Gamepad",
+                    "action": "Pause",
+                    "isComposite": false,
+                    "isPartOfComposite": false
+                },
                 {
                     "name": "",
                     "id": "0328a504-56c9-413d-b17e-f54c8515dee2",
@@ -775,6 +786,17 @@
                     "isComposite": false,
                     "isPartOfComposite": false
                 },
+                {
+                    "name": "",
+                    "id": "b2c3d4e5-f6a7-8b9c-0d1e-2f3a4b5c6d7f",
+                    "path": "<SwitchProControllerHID>/start",
+                    "interactions": "",
+                    "processors": "",
+                    "groups": ";Gamepad",
+                    "action": "Pause",
+                    "isComposite": false,
+                    "isPartOfComposite": false
+                },
                 {
                     "name": "",
                     "id": "9e92bb26-7e3b-4ec4-b06b-3c8f8e498ddc",