FIX: Properties of actions embedded into components not being editable (#766).

Author: Rene Damm

Packages/com.unity.inputsystem/CHANGELOG.md

@@ -20,6 +20,7 @@ however, it has to be formatted properly to pass verification tests.
 - It is now possible to use an empty binding path with a non empty override path.
 - It is now possible to use set an empty override path to disable a binding.
 - It is not possible to query the effectively used path of a binding using `effectivePath`.
+- Actions embedded into MonoBehaviour components can now have their properties edited in the inspector. Previously there was no way to get to the properties in this workflow. There is a gear icon now on the action that will open the action properties.
 
 ### Changed
 ### Added

Packages/com.unity.inputsystem/InputSystem/Editor/AssetEditor/InputActionTreeView.cs

@@ -1099,8 +1099,17 @@ private void DrawHeader(ref Rect rect)
                             AddNewActionMap();
                         }
                     }
+
+                    buttonRect.x -= buttonRect.width + EditorGUIUtility.standardVerticalSpacing;
                 }
             }
+
+            // Draw action properties button.
+            if (drawActionPropertiesButton && rootItem is ActionTreeItem item)
+            {
+                if (GUI.Button(buttonRect, s_ActionPropertiesIcon, GUIStyle.none))
+                    onDoubleClick?.Invoke(item);
+            }
         }
 
         // For each item, we draw
@@ -1251,6 +1260,7 @@ private bool ReloadIfSerializedObjectHasBeenChanged()
         public bool drawHeader { get; set; }
         public bool drawPlusButton { get; set; }
         public bool drawMinusButton { get; set; }
+        public bool drawActionPropertiesButton { get; set; }
         public float foldoutOffset { get; set; }
 
         public Action<SerializedProperty> onHandleAddNewAction { get; set; }
@@ -1309,6 +1319,7 @@ private GUIContent plusIcon
         private static readonly GUIContent s_PlusActionIcon = EditorGUIUtility.TrIconContent("Toolbar Plus", "Add Action");
         private static readonly GUIContent s_PlusActionMapIcon = EditorGUIUtility.TrIconContent("Toolbar Plus", "Add Action Map");
         private static readonly GUIContent s_DeleteSectionIcon = EditorGUIUtility.TrIconContent("Toolbar Minus", "Delete Selection");
+        private static readonly GUIContent s_ActionPropertiesIcon = EditorGUIUtility.TrIconContent("Settings", "Action Properties");
 
         private static readonly GUIContent s_CutLabel = EditorGUIUtility.TrTextContent("Cut");
         private static readonly GUIContent s_CopyLabel = EditorGUIUtility.TrTextContent("Copy");

Packages/com.unity.inputsystem/InputSystem/Editor/PropertyDrawers/InputActionDrawerBase.cs

@@ -48,7 +48,8 @@ private void InitTreeIfNeeded(SerializedProperty property)
                     title = property.displayName,
                     // With the tree in the inspector, the foldouts are drawn too far to the left. I don't
                     // really know where this is coming from. This works around it by adding an arbitrary offset...
-                    foldoutOffset = 14
+                    foldoutOffset = 14,
+                    drawActionPropertiesButton = true
                 };
                 m_TreeView.Reload();
             }