From c9b5193e51df9ec891c07bb321adfb82bd8d5331 Mon Sep 17 00:00:00 2001 From: Johannes Deml Date: Wed, 7 Feb 2024 19:31:05 +0100 Subject: [PATCH] Fix use prefixLabel position right away --- .../Editor/SubclassSelectorDrawer.cs | 17 ++++++----------- 1 file changed, 6 insertions(+), 11 deletions(-) diff --git a/Assets/MackySoft/MackySoft.SerializeReferenceExtensions/Editor/SubclassSelectorDrawer.cs b/Assets/MackySoft/MackySoft.SerializeReferenceExtensions/Editor/SubclassSelectorDrawer.cs index 2a612ed..0bfe2be 100644 --- a/Assets/MackySoft/MackySoft.SerializeReferenceExtensions/Editor/SubclassSelectorDrawer.cs +++ b/Assets/MackySoft/MackySoft.SerializeReferenceExtensions/Editor/SubclassSelectorDrawer.cs @@ -35,19 +35,14 @@ 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; - popupPosition.x += EditorGUIUtility.labelWidth; - popupPosition.height = EditorGUIUtility.singleLineHeight; + Rect popupPosition = EditorGUI.PrefixLabel(foldoutLabelRect, label); + // Draw the subclass selector popup. if (EditorGUI.DropdownButton(popupPosition,GetTypeName(property),FocusType.Keyboard)) { TypePopupCache popup = GetTypePopup(property); m_TargetProperty = property; @@ -62,7 +57,7 @@ namespace MackySoft.SerializeReferenceExtensions.Editor Rect foldoutRect = new Rect(position); foldoutRect.height = EditorGUIUtility.singleLineHeight; property.isExpanded = EditorGUI.Foldout(foldoutRect, property.isExpanded, GUIContent.none, true); - + if (property.isExpanded) { using (new EditorGUI.IndentLevelScope()) @@ -102,7 +97,7 @@ namespace MackySoft.SerializeReferenceExtensions.Editor if (!m_TypePopups.TryGetValue(managedReferenceFieldTypename,out TypePopupCache result)) { var state = new AdvancedDropdownState(); - + Type baseType = ManagedReferenceUtility.GetType(managedReferenceFieldTypename); var popup = new AdvancedTypePopup( TypeCache.GetTypesDerivedFrom(baseType).Append(baseType).Where(p => @@ -125,7 +120,7 @@ namespace MackySoft.SerializeReferenceExtensions.Editor object obj = individualProperty.SetManagedReference(type); individualProperty.isExpanded = (obj != null); - + individualObject.ApplyModifiedProperties(); individualObject.Update(); }