From a8bcece352d968080e4a1aa2f3e9659ffe49a741 Mon Sep 17 00:00:00 2001 From: Johannes Deml Date: Wed, 17 Jan 2024 13:37:32 +0100 Subject: [PATCH] Fix label overlapping button By drawing the label first we avoid the overlap. By drawing just the label without the foldout we need to work around a unity problem with indentation and apply that ourselves. The foldout property will then be rendered without gui content --- .../Editor/SubclassSelectorDrawer.cs | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/Assets/MackySoft/MackySoft.SerializeReferenceExtensions/Editor/SubclassSelectorDrawer.cs b/Assets/MackySoft/MackySoft.SerializeReferenceExtensions/Editor/SubclassSelectorDrawer.cs index aba3b02..2a612ed 100644 --- a/Assets/MackySoft/MackySoft.SerializeReferenceExtensions/Editor/SubclassSelectorDrawer.cs +++ b/Assets/MackySoft/MackySoft.SerializeReferenceExtensions/Editor/SubclassSelectorDrawer.cs @@ -35,6 +35,13 @@ namespace MackySoft.SerializeReferenceExtensions.Editor EditorGUI.BeginProperty(position,label,property); if (property.propertyType == SerializedPropertyType.ManagedReference) { + + // render label first to avoid label overlap for lists + Rect foldoutLabelRect = new Rect(position); + foldoutLabelRect.height = EditorGUIUtility.singleLineHeight; + foldoutLabelRect.x += EditorGUI.indentLevel * 12; + EditorGUI.PrefixLabel(foldoutLabelRect, label); + // Draw the subclass selector popup. Rect popupPosition = new Rect(position); popupPosition.width -= EditorGUIUtility.labelWidth; @@ -54,7 +61,7 @@ namespace MackySoft.SerializeReferenceExtensions.Editor // Draw the property with custom property drawer. Rect foldoutRect = new Rect(position); foldoutRect.height = EditorGUIUtility.singleLineHeight; - property.isExpanded = EditorGUI.Foldout(foldoutRect, property.isExpanded, label, true); + property.isExpanded = EditorGUI.Foldout(foldoutRect, property.isExpanded, GUIContent.none, true); if (property.isExpanded) { @@ -70,7 +77,7 @@ namespace MackySoft.SerializeReferenceExtensions.Editor } else { - EditorGUI.PropertyField(position, property, label, true); + EditorGUI.PropertyField(position, property, GUIContent.none, true); } } else { EditorGUI.LabelField(position,label,k_IsNotManagedReferenceLabel);