FIX: Avoid disabling project-wide actions on shutdown

Assets/Tests/InputSystem/Plugins/InputForUITests.cs

@@ -37,6 +37,7 @@ public class InputForUITests : InputTestFixture
     readonly List<Event> m_InputForUIEvents = new List<Event>();
     private int m_CurrentInputEventToCheck;
     InputSystemProvider m_InputSystemProvider;
+    private bool m_ClearedMockProvider;
 
     private InputActionAsset storedActions;
 
@@ -45,6 +46,7 @@ public override void Setup()
     {
         base.Setup();
         m_CurrentInputEventToCheck = 0;
+        m_ClearedMockProvider = false;
 
         storedActions = InputSystem.actions;
 
@@ -59,7 +61,8 @@ public override void Setup()
     public override void TearDown()
     {
         EventProvider.Unsubscribe(InputForUIOnEvent);
-        EventProvider.ClearMockProvider();
+        if (!m_ClearedMockProvider)
+            EventProvider.ClearMockProvider();
         m_InputForUIEvents.Clear();
 
         InputSystem.manager.actions = storedActions;
@@ -92,6 +95,38 @@ public void InputSystemActionAssetIsNotNull()
             "Test is invalid since InputSystemProvider actions are not available");
     }
 
+    [Test]
+    [Category(kTestCategory)]
+    public void Shutdown_DoesNotDisableProjectWideActionsAsset()
+    {
+        var asset = ScriptableObject.CreateInstance<InputActionAsset>();
+        var uiMap = new InputActionMap("UI");
+        uiMap.AddAction("Point", InputActionType.PassThrough, "<Mouse>/position");
+        uiMap.AddAction("Navigate", InputActionType.PassThrough, "<Gamepad>/leftStick");
+        uiMap.AddAction("Submit", InputActionType.Button, "<Keyboard>/enter");
+        uiMap.AddAction("Cancel", InputActionType.Button, "<Keyboard>/escape");
+        uiMap.AddAction("Click", InputActionType.PassThrough, "<Mouse>/leftButton");
+        uiMap.AddAction("MiddleClick", InputActionType.PassThrough, "<Mouse>/middleButton");
+        uiMap.AddAction("RightClick", InputActionType.PassThrough, "<Mouse>/rightButton");
+        uiMap.AddAction("ScrollWheel", InputActionType.PassThrough, "<Mouse>/scroll");
+        asset.AddActionMap(uiMap);
+
+        InputSystem.manager.actions = asset;
+        try
+        {
+            m_InputSystemProvider.Initialize();
+            Assert.That(uiMap.enabled, Is.True, "UI action map should be enabled by provider initialization.");
+
+            EventProvider.ClearMockProvider();
+            m_ClearedMockProvider = true;
+            Assert.That(uiMap.enabled, Is.True, "UI action map must remain enabled after provider shutdown.");
+        }
+        finally
+        {
+            Object.DestroyImmediate(asset);
+        }
+    }
+
     [Test]
     [Category(kTestCategory)]
     // Checks that mouse events are ignored when a touch is active.

Packages/com.unity.inputsystem/CHANGELOG.md

@@ -9,6 +9,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 
 ### Fixed
 
+- Fixed `InputSystemProvider` disabling project-wide actions on shutdown when UI Toolkit destroys its objects mid-play. The provider now scopes its lifecycle to the UI action map only and does not disable project-wide actions [UUM-134130](https://issuetracker.unity3d.com/product/unity/issues/guid/UUM-134130)
 - Fixed `InputActionRebindingExtensions.GetBindingDisplayString(InputAction, InputBinding, ...)` returning an empty string for composite bindings when the binding mask filters by group [UUM-141423](https://issuetracker.unity3d.com/product/unity/issues/guid/UUM-141423)
 - Fixed `InputEventPtr.handled` not preventing actions from triggering when switching devices. The default event handled policy has been changed from `SuppressStateUpdates` (now deprecated) to `SuppressActionEventNotifications`, which keeps device state synchronized while suppressing action callbacks for handled events. [ISXB-1097](https://issuetracker.unity3d.com/product/unity/issues/guid/ISXB-1097)
 - Fixed all `InputAction.WasXxxThisFrame()` and `WasXxxThisDynamicUpdate()` polling APIs to use per-action suppression tracking instead of a map-wide flag. Previously, when multiple events arrived in the same frame with mixed handled/unhandled states, the last event's suppression state could incorrectly affect all actions in the map. [ISXB-1097](https://issuetracker.unity3d.com/product/unity/issues/guid/ISXB-1097)

Packages/com.unity.inputsystem/InputSystem/Runtime/Plugins/InputForUI/InputSystemProvider.cs

@@ -17,6 +17,8 @@ internal class InputSystemProvider : IEventProviderImpl
 
         DefaultInputActions m_DefaultInputActions;
         InputActionAsset m_InputActionAsset;
+        InputActionMap m_UIActionMap;
+        bool m_ShouldDisableUIActionMapOnUnregister;
 
         // Note that these are plain action references instead since InputActionReference do
         // not provide any value when this integration doesn't have any UI. If this integration
@@ -636,14 +638,14 @@ void RegisterActions()
             m_RightClickAction = FindActionAndRegisterCallback(Actions.RightClickAction, OnRightClickPerformed);
             m_ScrollWheelAction = FindActionAndRegisterCallback(Actions.ScrollWheelAction, OnScrollWheelPerformed);
 
-            // When adding new actions, don't forget to add them to UnregisterActions
-            if (InputSystem.actions == null)
+            // Only touch the UI map so we don't change the enabled state of unrelated maps.
+            m_UIActionMap = m_InputActionAsset?.FindActionMap("UI", false);
+            if (m_UIActionMap != null && !m_UIActionMap.enabled)
             {
-                // If we've not loaded a user-created set of actions, just enable the UI actions from our defaults.
-                m_InputActionAsset.FindActionMap("UI", true).Enable();
+                // Don't take ownership of the UI map lifecycle for project-wide actions — the user manages those.
+                m_ShouldDisableUIActionMapOnUnregister = m_InputActionAsset != InputSystem.actions;
+                m_UIActionMap.Enable();
             }
-            else
-                m_InputActionAsset.Enable();
         }
 
         void UnregisterAction(ref InputAction action, Action<InputAction.CallbackContext> callback = null)
@@ -664,8 +666,11 @@ void UnregisterActions()
             UnregisterAction(ref m_RightClickAction, OnRightClickPerformed);
             UnregisterAction(ref m_ScrollWheelAction, OnScrollWheelPerformed);
 
-            if (m_InputActionAsset != null)
-                m_InputActionAsset.Disable();
+            if (m_ShouldDisableUIActionMapOnUnregister && m_UIActionMap != null)
+                m_UIActionMap.Disable();
+
+            m_UIActionMap = null;
+            m_ShouldDisableUIActionMapOnUnregister = false;
         }
 
         void SelectInputActionAsset()