Added a settings that detect imgui rendering to force it even if an uitoolkit implementation exist

bmalrat · bmalrat · commit 0dde2b824404 · 2024-12-03T15:43:11.000-05:00
diff --git a/Packages/com.unity.inputsystem/InputSystem/Actions/Composites/AxisComposite.cs b/Packages/com.unity.inputsystem/InputSystem/Actions/Composites/AxisComposite.cs
@@ -220,7 +220,7 @@ internal class AxisCompositeEditor : InputParameterEditor<AxisComposite>
         public override void OnGUI()
         {
 #if UNITY_INPUT_SYSTEM_PROJECT_WIDE_ACTIONS
-            if (!InputSystem.settings.IsFeatureEnabled(InputFeatureNames.kUseIMGUIEditorForAssets)) return;
+            if (!InputSystem.settings.useIMGUIEditorForAssets) return;
 #endif
             target.whichSideWins = (AxisComposite.WhichSideWins)EditorGUILayout.EnumPopup(m_WhichAxisWinsLabel, target.whichSideWins);
         }
diff --git a/Packages/com.unity.inputsystem/InputSystem/Actions/Composites/Vector2Composite.cs b/Packages/com.unity.inputsystem/InputSystem/Actions/Composites/Vector2Composite.cs
@@ -200,7 +200,7 @@ internal class Vector2CompositeEditor : InputParameterEditor<Vector2Composite>
         public override void OnGUI()
         {
 #if UNITY_INPUT_SYSTEM_PROJECT_WIDE_ACTIONS
-            if (!InputSystem.settings.IsFeatureEnabled(InputFeatureNames.kUseIMGUIEditorForAssets)) return;
+            if (!InputSystem.settings.useIMGUIEditorForAssets) return;
 #endif
             target.mode = (Vector2Composite.Mode)EditorGUILayout.EnumPopup(m_ModeLabel, target.mode);
         }
diff --git a/Packages/com.unity.inputsystem/InputSystem/Actions/Composites/Vector3Composite.cs b/Packages/com.unity.inputsystem/InputSystem/Actions/Composites/Vector3Composite.cs
@@ -180,7 +180,7 @@ internal class Vector3CompositeEditor : InputParameterEditor<Vector3Composite>
         public override void OnGUI()
         {
 #if UNITY_INPUT_SYSTEM_PROJECT_WIDE_ACTIONS
-            if (!InputSystem.settings.IsFeatureEnabled(InputFeatureNames.kUseIMGUIEditorForAssets)) return;
+            if (!InputSystem.settings.useIMGUIEditorForAssets) return;
 #endif
             target.mode = (Vector3Composite.Mode)EditorGUILayout.EnumPopup(m_ModeLabel, target.mode);
         }
diff --git a/Packages/com.unity.inputsystem/InputSystem/Actions/Interactions/HoldInteraction.cs b/Packages/com.unity.inputsystem/InputSystem/Actions/Interactions/HoldInteraction.cs
@@ -125,7 +125,7 @@ protected override void OnEnable()
         public override void OnGUI()
         {
 #if UNITY_INPUT_SYSTEM_PROJECT_WIDE_ACTIONS
-            if (!InputSystem.settings.IsFeatureEnabled(InputFeatureNames.kUseIMGUIEditorForAssets)) return;
+            if (!InputSystem.settings.useIMGUIEditorForAssets) return;
 #endif
             m_PressPointSetting.OnGUI();
             m_DurationSetting.OnGUI();
diff --git a/Packages/com.unity.inputsystem/InputSystem/Actions/Interactions/MultiTapInteraction.cs b/Packages/com.unity.inputsystem/InputSystem/Actions/Interactions/MultiTapInteraction.cs
@@ -197,7 +197,7 @@ protected override void OnEnable()
         public override void OnGUI()
         {
 #if UNITY_INPUT_SYSTEM_PROJECT_WIDE_ACTIONS
-            if (!InputSystem.settings.IsFeatureEnabled(InputFeatureNames.kUseIMGUIEditorForAssets)) return;
+            if (!InputSystem.settings.useIMGUIEditorForAssets) return;
 #endif
             target.tapCount = EditorGUILayout.IntField(m_TapCountLabel, target.tapCount);
             m_TapDelaySetting.OnGUI();
diff --git a/Packages/com.unity.inputsystem/InputSystem/Actions/Interactions/PressInteraction.cs b/Packages/com.unity.inputsystem/InputSystem/Actions/Interactions/PressInteraction.cs
@@ -214,7 +214,7 @@ protected override void OnEnable()
         public override void OnGUI()
         {
 #if UNITY_INPUT_SYSTEM_PROJECT_WIDE_ACTIONS
-            if (!InputSystem.settings.IsFeatureEnabled(InputFeatureNames.kUseIMGUIEditorForAssets)) return;
+            if (!InputSystem.settings.useIMGUIEditorForAssets) return;
 #endif
             EditorGUILayout.HelpBox(s_HelpBoxText);
             target.behavior = (PressBehavior)EditorGUILayout.EnumPopup(s_PressBehaviorLabel, target.behavior);
diff --git a/Packages/com.unity.inputsystem/InputSystem/Actions/Interactions/SlowTapInteraction.cs b/Packages/com.unity.inputsystem/InputSystem/Actions/Interactions/SlowTapInteraction.cs
@@ -89,7 +89,7 @@ protected override void OnEnable()
         public override void OnGUI()
         {
 #if UNITY_INPUT_SYSTEM_PROJECT_WIDE_ACTIONS
-            if (!InputSystem.settings.IsFeatureEnabled(InputFeatureNames.kUseIMGUIEditorForAssets)) return;
+            if (!InputSystem.settings.useIMGUIEditorForAssets) return;
 #endif
             m_DurationSetting.OnGUI();
             m_PressPointSetting.OnGUI();
diff --git a/Packages/com.unity.inputsystem/InputSystem/Actions/Interactions/TapInteraction.cs b/Packages/com.unity.inputsystem/InputSystem/Actions/Interactions/TapInteraction.cs
@@ -103,7 +103,7 @@ protected override void OnEnable()
         public override void OnGUI()
         {
 #if UNITY_INPUT_SYSTEM_PROJECT_WIDE_ACTIONS
-            if (!InputSystem.settings.IsFeatureEnabled(InputFeatureNames.kUseIMGUIEditorForAssets)) return;
+            if (!InputSystem.settings.useIMGUIEditorForAssets) return;
 #endif
             m_DurationSetting.OnGUI();
             m_PressPointSetting.OnGUI();
diff --git a/Packages/com.unity.inputsystem/InputSystem/Controls/Processors/AxisDeadzoneProcessor.cs b/Packages/com.unity.inputsystem/InputSystem/Controls/Processors/AxisDeadzoneProcessor.cs
@@ -94,7 +94,7 @@ protected override void OnEnable()
         public override void OnGUI()
         {
 #if UNITY_INPUT_SYSTEM_PROJECT_WIDE_ACTIONS
-            if (!InputSystem.settings.IsFeatureEnabled(InputFeatureNames.kUseIMGUIEditorForAssets)) return;
+            if (!InputSystem.settings.useIMGUIEditorForAssets) return;
 #endif
             m_MinSetting.OnGUI();
             m_MaxSetting.OnGUI();
diff --git a/Packages/com.unity.inputsystem/InputSystem/Controls/Processors/StickDeadzoneProcessor.cs b/Packages/com.unity.inputsystem/InputSystem/Controls/Processors/StickDeadzoneProcessor.cs
@@ -83,7 +83,7 @@ protected override void OnEnable()
         public override void OnGUI()
         {
 #if UNITY_INPUT_SYSTEM_PROJECT_WIDE_ACTIONS
-            if (!InputSystem.settings.IsFeatureEnabled(InputFeatureNames.kUseIMGUIEditorForAssets)) return;
+            if (!InputSystem.settings.useIMGUIEditorForAssets) return;
 #endif
             m_MinSetting.OnGUI();
             m_MaxSetting.OnGUI();
diff --git a/Packages/com.unity.inputsystem/InputSystem/Editor/AssetEditor/InputActionEditorWindow.cs b/Packages/com.unity.inputsystem/InputSystem/Editor/AssetEditor/InputActionEditorWindow.cs
@@ -45,7 +45,7 @@ static InputActionEditorWindow()
         public static bool OnOpenAsset(int instanceId, int line)
         {
 #if UNITY_INPUT_SYSTEM_PROJECT_WIDE_ACTIONS
-            if (!InputSystem.settings.IsFeatureEnabled(InputFeatureNames.kUseIMGUIEditorForAssets))
+            if (!InputSystem.settings.useIMGUIEditorForAssets)
                 return false;
 #endif
             var path = AssetDatabase.GetAssetPath(instanceId);
diff --git a/Packages/com.unity.inputsystem/InputSystem/Editor/AssetEditor/ParameterListView.cs b/Packages/com.unity.inputsystem/InputSystem/Editor/AssetEditor/ParameterListView.cs
@@ -353,7 +353,7 @@ public void OnGUI()
 
 #if UNITY_INPUT_SYSTEM_PROJECT_WIDE_ACTIONS
             // handled by OnDrawVisualElements with UI Toolkit
-            if (!InputSystem.settings.IsFeatureEnabled(InputFeatureNames.kUseIMGUIEditorForAssets)) return;
+            if (!InputSystem.settings.useIMGUIEditorForAssets) return;
 #endif
             // Otherwise, fall back to our default logic.
             if (m_Parameters == null)
diff --git a/Packages/com.unity.inputsystem/InputSystem/Editor/AssetImporter/InputActionImporterEditor.cs b/Packages/com.unity.inputsystem/InputSystem/Editor/AssetImporter/InputActionImporterEditor.cs
@@ -150,7 +150,7 @@ private static void OpenEditor(InputActionAsset asset)
             }
 
             // Redirect to UI-Toolkit window editor if not configured to use IMGUI explicitly
-            if (!InputSystem.settings.IsFeatureEnabled(InputFeatureNames.kUseIMGUIEditorForAssets))
+            if (!InputSystem.settings.useIMGUIEditorForAssets)
                 InputActionsEditorWindow.OpenEditor(asset);
             else
                 InputActionEditorWindow.OpenEditor(asset);
diff --git a/Packages/com.unity.inputsystem/InputSystem/InputSettings.cs b/Packages/com.unity.inputsystem/InputSystem/InputSettings.cs
@@ -978,6 +978,10 @@ public enum InputActionPropertyDrawerMode
             MultilineBoth,
         }
 
+#if UNITY_EDITOR && UNITY_INPUT_SYSTEM_PROJECT_WIDE_ACTIONS
+        public bool useIMGUIEditorForAssets => UnityEditor.EditorGUI.indentLevel >0 || IsFeatureEnabled(InputFeatureNames.kUseIMGUIEditorForAssets);
+#endif
+
         private static bool CompareFloats(float a, float b)
         {
             return (a - b) <= float.Epsilon;

Original file line number	Diff line number	Diff line change
`@@ -220,7 +220,7 @@ internal class AxisCompositeEditor : InputParameterEditor<AxisComposite>`
`220`	`220`	`public override void OnGUI()`
`221`	`221`	`{`
`222`	`222`	`#if UNITY_INPUT_SYSTEM_PROJECT_WIDE_ACTIONS`
`223`		`- if (!InputSystem.settings.IsFeatureEnabled(InputFeatureNames.kUseIMGUIEditorForAssets)) return;`
	`223`	`+ if (!InputSystem.settings.useIMGUIEditorForAssets) return;`
`224`	`224`	`#endif`
`225`	`225`	`target.whichSideWins = (AxisComposite.WhichSideWins)EditorGUILayout.EnumPopup(m_WhichAxisWinsLabel, target.whichSideWins);`
`226`	`226`	`}`
Original file line number	Diff line number	Diff line change
`@@ -200,7 +200,7 @@ internal class Vector2CompositeEditor : InputParameterEditor<Vector2Composite>`
`200`	`200`	`public override void OnGUI()`
`201`	`201`	`{`
`202`	`202`	`#if UNITY_INPUT_SYSTEM_PROJECT_WIDE_ACTIONS`
`203`		`- if (!InputSystem.settings.IsFeatureEnabled(InputFeatureNames.kUseIMGUIEditorForAssets)) return;`
	`203`	`+ if (!InputSystem.settings.useIMGUIEditorForAssets) return;`
`204`	`204`	`#endif`
`205`	`205`	`target.mode = (Vector2Composite.Mode)EditorGUILayout.EnumPopup(m_ModeLabel, target.mode);`
`206`	`206`	`}`
Original file line number	Diff line number	Diff line change
`@@ -180,7 +180,7 @@ internal class Vector3CompositeEditor : InputParameterEditor<Vector3Composite>`
`180`	`180`	`public override void OnGUI()`
`181`	`181`	`{`
`182`	`182`	`#if UNITY_INPUT_SYSTEM_PROJECT_WIDE_ACTIONS`
`183`		`- if (!InputSystem.settings.IsFeatureEnabled(InputFeatureNames.kUseIMGUIEditorForAssets)) return;`
	`183`	`+ if (!InputSystem.settings.useIMGUIEditorForAssets) return;`
`184`	`184`	`#endif`
`185`	`185`	`target.mode = (Vector3Composite.Mode)EditorGUILayout.EnumPopup(m_ModeLabel, target.mode);`
`186`	`186`	`}`
Original file line number	Diff line number	Diff line change
`@@ -125,7 +125,7 @@ protected override void OnEnable()`
`125`	`125`	`public override void OnGUI()`
`126`	`126`	`{`
`127`	`127`	`#if UNITY_INPUT_SYSTEM_PROJECT_WIDE_ACTIONS`
`128`		`- if (!InputSystem.settings.IsFeatureEnabled(InputFeatureNames.kUseIMGUIEditorForAssets)) return;`
	`128`	`+ if (!InputSystem.settings.useIMGUIEditorForAssets) return;`
`129`	`129`	`#endif`
`130`	`130`	`m_PressPointSetting.OnGUI();`
`131`	`131`	`m_DurationSetting.OnGUI();`
Original file line number	Diff line number	Diff line change
`@@ -197,7 +197,7 @@ protected override void OnEnable()`
`197`	`197`	`public override void OnGUI()`
`198`	`198`	`{`
`199`	`199`	`#if UNITY_INPUT_SYSTEM_PROJECT_WIDE_ACTIONS`
`200`		`- if (!InputSystem.settings.IsFeatureEnabled(InputFeatureNames.kUseIMGUIEditorForAssets)) return;`
	`200`	`+ if (!InputSystem.settings.useIMGUIEditorForAssets) return;`
`201`	`201`	`#endif`
`202`	`202`	`target.tapCount = EditorGUILayout.IntField(m_TapCountLabel, target.tapCount);`
`203`	`203`	`m_TapDelaySetting.OnGUI();`
Original file line number	Diff line number	Diff line change
`@@ -214,7 +214,7 @@ protected override void OnEnable()`
`214`	`214`	`public override void OnGUI()`
`215`	`215`	`{`
`216`	`216`	`#if UNITY_INPUT_SYSTEM_PROJECT_WIDE_ACTIONS`
`217`		`- if (!InputSystem.settings.IsFeatureEnabled(InputFeatureNames.kUseIMGUIEditorForAssets)) return;`
	`217`	`+ if (!InputSystem.settings.useIMGUIEditorForAssets) return;`
`218`	`218`	`#endif`
`219`	`219`	`EditorGUILayout.HelpBox(s_HelpBoxText);`
`220`	`220`	`target.behavior = (PressBehavior)EditorGUILayout.EnumPopup(s_PressBehaviorLabel, target.behavior);`
Original file line number	Diff line number	Diff line change
`@@ -89,7 +89,7 @@ protected override void OnEnable()`
`89`	`89`	`public override void OnGUI()`
`90`	`90`	`{`
`91`	`91`	`#if UNITY_INPUT_SYSTEM_PROJECT_WIDE_ACTIONS`
`92`		`- if (!InputSystem.settings.IsFeatureEnabled(InputFeatureNames.kUseIMGUIEditorForAssets)) return;`
	`92`	`+ if (!InputSystem.settings.useIMGUIEditorForAssets) return;`
`93`	`93`	`#endif`
`94`	`94`	`m_DurationSetting.OnGUI();`
`95`	`95`	`m_PressPointSetting.OnGUI();`
Original file line number	Diff line number	Diff line change
`@@ -103,7 +103,7 @@ protected override void OnEnable()`
`103`	`103`	`public override void OnGUI()`
`104`	`104`	`{`
`105`	`105`	`#if UNITY_INPUT_SYSTEM_PROJECT_WIDE_ACTIONS`
`106`		`- if (!InputSystem.settings.IsFeatureEnabled(InputFeatureNames.kUseIMGUIEditorForAssets)) return;`
	`106`	`+ if (!InputSystem.settings.useIMGUIEditorForAssets) return;`
`107`	`107`	`#endif`
`108`	`108`	`m_DurationSetting.OnGUI();`
`109`	`109`	`m_PressPointSetting.OnGUI();`
Original file line number	Diff line number	Diff line change
`@@ -94,7 +94,7 @@ protected override void OnEnable()`
`94`	`94`	`public override void OnGUI()`
`95`	`95`	`{`
`96`	`96`	`#if UNITY_INPUT_SYSTEM_PROJECT_WIDE_ACTIONS`
`97`		`- if (!InputSystem.settings.IsFeatureEnabled(InputFeatureNames.kUseIMGUIEditorForAssets)) return;`
	`97`	`+ if (!InputSystem.settings.useIMGUIEditorForAssets) return;`
`98`	`98`	`#endif`
`99`	`99`	`m_MinSetting.OnGUI();`
`100`	`100`	`m_MaxSetting.OnGUI();`
Original file line number	Diff line number	Diff line change
`@@ -83,7 +83,7 @@ protected override void OnEnable()`
`83`	`83`	`public override void OnGUI()`
`84`	`84`	`{`
`85`	`85`	`#if UNITY_INPUT_SYSTEM_PROJECT_WIDE_ACTIONS`
`86`		`- if (!InputSystem.settings.IsFeatureEnabled(InputFeatureNames.kUseIMGUIEditorForAssets)) return;`
	`86`	`+ if (!InputSystem.settings.useIMGUIEditorForAssets) return;`
`87`	`87`	`#endif`
`88`	`88`	`m_MinSetting.OnGUI();`
`89`	`89`	`m_MaxSetting.OnGUI();`