Implement basic path validation for the SerializedDirectory class; minor refactor changes

arimger · arimger · commit 450f34068c80 · 2024-08-29T00:12:17.000+02:00
diff --git a/Assets/Editor Toolbox/Editor/Drawers/Internal/FolderDataDrawer.cs b/Assets/Editor Toolbox/Editor/Drawers/Internal/FolderDataDrawer.cs
@@ -11,7 +11,6 @@ internal class FolderDataDrawer : PropertyDrawer
         private const int largeIconPickedId = 1001;
         private const int smallIconPickedId = 1002;
 
-
         private void DrawFolderByNameIcon(Rect rect)
         {
             var diff = rect.height - Style.dataByNameLabelStyle.fixedHeight;
@@ -67,7 +66,6 @@ private int GetSelectorHash(SerializedProperty property)
             return property.propertyPath.GetHashCode() + property.serializedObject.GetHashCode();
         }
 
-
         public override float GetPropertyHeight(SerializedProperty property, GUIContent label)
         {
             //if expanded - draw foldout + 3 properties + buttons strip + additional icon 
@@ -154,35 +152,29 @@ public override void OnGUI(Rect position, SerializedProperty property, GUIConten
             EditorGUI.indentLevel++;
 
             var rawPropertyHeight = propertyPosition.height + Style.spacing;
-            var summaryFieldHeight = rawPropertyHeight;
 
             propertyPosition.y += rawPropertyHeight;
-            summaryFieldHeight += rawPropertyHeight;
             //draw the folder data type property
             EditorGUI.PropertyField(propertyPosition, dataTypeProperty, new GUIContent("Type"), false);
             propertyPosition.y += rawPropertyHeight;
-            summaryFieldHeight += rawPropertyHeight;
 
             //decide what property should be drawn depending on the folder data type
             if (isPathBased)
             {
                 propertyPosition.height = EditorGUI.GetPropertyHeight(pathProperty);
                 EditorGUI.PropertyField(propertyPosition, pathProperty, false);
                 propertyPosition.y += propertyPosition.height + Style.spacing;
-                summaryFieldHeight += propertyPosition.height + Style.spacing;
                 propertyPosition.height = Style.height;
             }
             else
             {
                 EditorGUI.PropertyField(propertyPosition, nameProperty, false);
                 propertyPosition.y += rawPropertyHeight;
-                summaryFieldHeight += rawPropertyHeight;
             }
 
             propertyPosition.height = EditorGUI.GetPropertyHeight(tooltipProperty);
             EditorGUI.PropertyField(propertyPosition, tooltipProperty, false);
             propertyPosition.y += propertyPosition.height + Style.spacing;
-            summaryFieldHeight += propertyPosition.height + Style.spacing;
             propertyPosition.height = Style.height;
 
             //adjust rect for folder icons + button strip
@@ -244,7 +236,6 @@ public override void OnGUI(Rect position, SerializedProperty property, GUIConten
             EditorGUI.EndProperty();
         }
 
-
         private static class Style
         {
             internal static readonly float height = EditorGUIUtility.singleLineHeight;
diff --git a/Assets/Editor Toolbox/Editor/Drawers/Material/BaseMaterialPropertyDrawer.cs b/Assets/Editor Toolbox/Editor/Drawers/Material/BaseMaterialPropertyDrawer.cs
@@ -29,15 +29,14 @@ protected virtual void DrawInvalidDrawerLabel(Rect position, string label)
             EditorGUI.LabelField(position, content);
         }
 
-
-        public override sealed float GetPropertyHeight(MaterialProperty prop, string label, MaterialEditor editor)
+        public sealed override float GetPropertyHeight(MaterialProperty prop, string label, MaterialEditor editor)
         {
             return IsPropertyValid(prop)
                 ? GetPropertyHeightSafe(prop, label, editor)
                 : base.GetPropertyHeight(prop, label, editor);
         }
 
-        public override sealed void OnGUI(Rect position, MaterialProperty prop, string label, MaterialEditor editor)
+        public sealed override void OnGUI(Rect position, MaterialProperty prop, string label, MaterialEditor editor)
         {
             if (IsPropertyValid(prop))
             {
diff --git a/Assets/Editor Toolbox/Editor/Drawers/Material/MaterialCompactTextureDrawer.cs b/Assets/Editor Toolbox/Editor/Drawers/Material/MaterialCompactTextureDrawer.cs
@@ -7,7 +7,6 @@ public class MaterialCompactTextureDrawer : BaseMaterialPropertyDrawer
     {
         private readonly string tooltip;
 
-
         public MaterialCompactTextureDrawer() : this(string.Empty)
         { }
 
@@ -16,7 +15,6 @@ public MaterialCompactTextureDrawer(string tooltip)
             this.tooltip = tooltip;
         }
 
-
         protected override float GetPropertyHeightSafe(MaterialProperty prop, string label, MaterialEditor editor)
         {
             return EditorGUIUtility.singleLineHeight;
diff --git a/Assets/Editor Toolbox/Editor/Drawers/Material/MaterialConditionalDrawer.cs b/Assets/Editor Toolbox/Editor/Drawers/Material/MaterialConditionalDrawer.cs
@@ -7,13 +7,11 @@ public abstract class MaterialConditionalDrawer : BaseMaterialPropertyDrawer
     {
         protected readonly string togglePropertyName;
 
-
         protected MaterialConditionalDrawer(string togglePropertyName)
         {
             this.togglePropertyName = togglePropertyName;
         }
 
-
         protected override float GetPropertyHeightSafe(MaterialProperty prop, string label, MaterialEditor editor)
         {
             if (!HasToggle(prop))
diff --git a/Assets/Editor Toolbox/Editor/Drawers/Material/MaterialHelpDecorator.cs b/Assets/Editor Toolbox/Editor/Drawers/Material/MaterialHelpDecorator.cs
@@ -8,7 +8,6 @@ public class MaterialHelpDecorator : BaseMaterialPropertyDrawer
         private readonly string message;
         private readonly MessageType type;
 
-
         public MaterialHelpDecorator(string message) : this(message, 1)
         {
             this.message = message;
@@ -20,7 +19,6 @@ public MaterialHelpDecorator(string message, float type)
             this.type = (MessageType)type;
         }
 
-
         protected override float GetPropertyHeightSafe(MaterialProperty prop, string label, MaterialEditor editor)
         {
             var minHeight = 24.0f;
@@ -33,7 +31,6 @@ protected override void OnGUISafe(Rect position, MaterialProperty prop, string l
             EditorGUI.HelpBox(position, message, type);
         }
 
-
         private static class Style
         {
             internal static readonly GUIStyle textStyle = new GUIStyle(EditorStyles.label)
diff --git a/Assets/Editor Toolbox/Editor/Drawers/Material/MaterialHideIfToggleDrawer.cs b/Assets/Editor Toolbox/Editor/Drawers/Material/MaterialHideIfToggleDrawer.cs
@@ -5,7 +5,6 @@ public class MaterialHideIfToggleDrawer : MaterialConditionalDrawer
         public MaterialHideIfToggleDrawer(string togglePropertyName) : base(togglePropertyName)
         { }
 
-
         protected override bool IsVisible(bool? value)
         {
             return value.HasValue && !value.Value;
diff --git a/Assets/Editor Toolbox/Editor/Drawers/Material/MaterialIndentDrawer.cs b/Assets/Editor Toolbox/Editor/Drawers/Material/MaterialIndentDrawer.cs
@@ -7,7 +7,6 @@ public class MaterialIndentDrawer : BaseMaterialPropertyDrawer
     {
         private readonly int indent;
 
-
         public MaterialIndentDrawer() : this(1)
         { }
 
@@ -16,7 +15,6 @@ public MaterialIndentDrawer(float indent)
             this.indent = (int)indent;
         }
 
-
         protected override float GetPropertyHeightSafe(MaterialProperty prop, string label, MaterialEditor editor)
         {
             return base.GetPropertyHeightSafe(prop, label, editor);
diff --git a/Assets/Editor Toolbox/Editor/Drawers/Material/MaterialMinMaxSliderDrawer.cs b/Assets/Editor Toolbox/Editor/Drawers/Material/MaterialMinMaxSliderDrawer.cs
@@ -10,14 +10,12 @@ public class MaterialMinMaxSliderDrawer : BaseMaterialPropertyDrawer
         private readonly float minValue;
         private readonly float maxValue;
 
-
         public MaterialMinMaxSliderDrawer(float minValue, float maxValue)
         {
             this.minValue = minValue;
             this.maxValue = maxValue;
         }
 
-
         protected override float GetPropertyHeightSafe(MaterialProperty prop, string label, MaterialEditor editor)
         {
             return EditorGUIUtility.singleLineHeight;
diff --git a/Assets/Editor Toolbox/Editor/Drawers/Material/MaterialShowIfToggleDrawer.cs b/Assets/Editor Toolbox/Editor/Drawers/Material/MaterialShowIfToggleDrawer.cs
@@ -5,7 +5,6 @@ public class MaterialShowIfToggleDrawer : MaterialConditionalDrawer
         public MaterialShowIfToggleDrawer(string togglePropertyName) : base(togglePropertyName)
         { }
 
-
         protected override bool IsVisible(bool? value)
         {
             return value ?? false;
diff --git a/Assets/Editor Toolbox/Editor/Drawers/Material/MaterialTitleDecorator.cs b/Assets/Editor Toolbox/Editor/Drawers/Material/MaterialTitleDecorator.cs
@@ -8,7 +8,6 @@ public class MaterialTitleDecorator : BaseMaterialPropertyDrawer
         private readonly GUIContent header;
         private readonly float spacing;
 
-
         public MaterialTitleDecorator(string header) : this(header, 4.0f)
         { }
 
@@ -18,7 +17,6 @@ public MaterialTitleDecorator(string header, float spacing)
             this.spacing = spacing;
         }
 
-
         protected override float GetPropertyHeightSafe(MaterialProperty prop, string label, MaterialEditor editor)
         {
             return EditorGUIUtility.singleLineHeight + spacing;
diff --git a/Assets/Editor Toolbox/Editor/Drawers/Regular/SerializedDateTimeDrawer.cs b/Assets/Editor Toolbox/Editor/Drawers/Regular/SerializedDateTimeDrawer.cs
@@ -1,4 +1,5 @@
-using System;
+﻿using System;
+
 using UnityEditor;
 using UnityEngine;
 
@@ -12,7 +13,7 @@ protected override void OnGUISafe(Rect position, SerializedProperty property, GU
             label = EditorGUI.BeginProperty(position, label, property);
             var fieldPosition = EditorGUI.PrefixLabel(position, label);
             var ticksProperty = property.FindPropertyRelative("ticks");
-            DateTime dateTime = new DateTime(ticksProperty.longValue);
+            var dateTime = new DateTime(ticksProperty.longValue);
             EditorGUI.BeginChangeCheck();
             var dateTimeString = EditorGUI.DelayedTextField(fieldPosition, dateTime.ToString());
             if (EditorGUI.EndChangeCheck())
diff --git a/Assets/Editor Toolbox/Editor/Drawers/Regular/SerializedDirectoryDrawer.cs b/Assets/Editor Toolbox/Editor/Drawers/Regular/SerializedDirectoryDrawer.cs
@@ -0,0 +1,38 @@
+﻿using UnityEditor;
+using UnityEngine;
+
+namespace Toolbox.Editor.Drawers
+{
+    [CustomPropertyDrawer(typeof(SerializedDirectory))]
+    public class SerializedDirectoryDrawer : PropertyDrawerBase
+    {
+        protected override void OnGUISafe(Rect position, SerializedProperty property, GUIContent label)
+        {
+            var assetProperty = property.FindPropertyRelative("directoryAsset");
+            var pathProperty = property.FindPropertyRelative("path");
+
+            label = EditorGUI.BeginProperty(position, label, property);
+
+            position.yMax = position.yMin + EditorGUIUtility.singleLineHeight;
+            property.isExpanded = EditorGUI.Foldout(position, property.isExpanded, label);
+
+            if (property.isExpanded)
+            {
+                EditorGUI.indentLevel++;
+
+                position.y += EditorGUIUtility.singleLineHeight + EditorGUIUtility.standardVerticalSpacing;
+                EditorGUI.PropertyField(position, assetProperty);
+
+                position.y += EditorGUIUtility.singleLineHeight + EditorGUIUtility.standardVerticalSpacing;
+                using (new EditorGUI.DisabledScope(true))
+                {
+                    EditorGUI.PropertyField(position, pathProperty);
+                }
+
+                EditorGUI.indentLevel--;
+            }
+
+            EditorGUI.EndProperty();
+        }
+    }
+}
diff --git a/Assets/Editor Toolbox/Editor/Drawers/Regular/SerializedDirectoryDrawer.cs.meta b/Assets/Editor Toolbox/Editor/Drawers/Regular/SerializedDirectoryDrawer.cs.meta
diff --git a/Assets/Editor Toolbox/Runtime/Attributes/Property/Regular/TypeConstraintAttribute.cs b/Assets/Editor Toolbox/Runtime/Attributes/Property/Regular/TypeConstraintAttribute.cs
@@ -17,7 +17,6 @@ public TypeConstraintAttribute(Type assemblyType)
             AssemblyType = assemblyType;
         }
 
-
         /// <summary>
         /// Associated type which will define what type we are looking for.
         /// </summary>
diff --git a/Assets/Editor Toolbox/Runtime/Serialization/SerializedDirectory.cs b/Assets/Editor Toolbox/Runtime/Serialization/SerializedDirectory.cs
@@ -1,4 +1,5 @@
-using System;
+﻿using System;
+
 #if UNITY_EDITOR
 using UnityEditor;
 #endif
@@ -15,13 +16,38 @@ public class SerializedDirectory : ISerializationCallbackReceiver
         [SerializeField, Disable]
         private string path;
 
+        internal static bool IsAssetValid(DefaultAsset asset, out string path)
+        {
+            path = asset != null ? AssetDatabase.GetAssetPath(asset) : null;
+            if (path == null)
+            {
+                return true;
+            }
+
+            //NOTE: assume that directory can't have any extension
+            var extension = System.IO.Path.GetExtension(path);
+            if (string.IsNullOrEmpty(extension))
+            {
+                return true;
+            }
+
+            return false;
+        }
+
         void ISerializationCallbackReceiver.OnAfterDeserialize()
         { }
 
         void ISerializationCallbackReceiver.OnBeforeSerialize()
         {
 #if UNITY_EDITOR
-            path = directoryAsset ? AssetDatabase.GetAssetPath(directoryAsset) : null;
+            if (IsAssetValid(directoryAsset, out var assetPath))
+            {
+                DirectoryPath = assetPath;
+                return;
+            }
+
+            DirectoryAsset = null;
+            DirectoryPath = null;
 #endif
         }
 
@@ -32,6 +58,13 @@ public DefaultAsset DirectoryAsset
             set => directoryAsset = value;
         }
 #endif
+        public string DirectoryPath
+        {
+            get => path;
+            set => path = value;
+        }
+
+        [Obsolete("Use " + nameof(DirectoryPath) + " instead.")]
         public string Path
         {
             get => path;
diff --git a/Assets/Examples/Scenes/SampleScene.unity b/Assets/Examples/Scenes/SampleScene.unity
@@ -1816,7 +1816,11 @@ MonoBehaviour:
     scenePath: Assets/Examples/Scenes/SampleScene.unity
     buildIndex: 0
   dictionary:
-    pairs: []
+    pairs:
+    - key: 0
+      value: {fileID: 2037155952}
+    - key: 1
+      value: {fileID: 1972418676}
     error: 0
   date:
     ticks: 637820719200000000

Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,6 @@ internal class FolderDataDrawer : PropertyDrawer`
`11`	`11`	`private const int largeIconPickedId = 1001;`
`12`	`12`	`private const int smallIconPickedId = 1002;`
`13`	`13`
`14`		`-`
`15`	`14`	`private void DrawFolderByNameIcon(Rect rect)`
`16`	`15`	`{`
`17`	`16`	`var diff = rect.height - Style.dataByNameLabelStyle.fixedHeight;`
`@@ -67,7 +66,6 @@ private int GetSelectorHash(SerializedProperty property)`
`67`	`66`	`return property.propertyPath.GetHashCode() + property.serializedObject.GetHashCode();`
`68`	`67`	`}`
`69`	`68`
`70`		`-`
`71`	`69`	`public override float GetPropertyHeight(SerializedProperty property, GUIContent label)`
`72`	`70`	`{`
`73`	`71`	`//if expanded - draw foldout + 3 properties + buttons strip + additional icon`
`@@ -154,35 +152,29 @@ public override void OnGUI(Rect position, SerializedProperty property, GUIConten`
`154`	`152`	`EditorGUI.indentLevel++;`
`155`	`153`
`156`	`154`	`var rawPropertyHeight = propertyPosition.height + Style.spacing;`
`157`		`- var summaryFieldHeight = rawPropertyHeight;`
`158`	`155`
`159`	`156`	`propertyPosition.y += rawPropertyHeight;`
`160`		`- summaryFieldHeight += rawPropertyHeight;`
`161`	`157`	`//draw the folder data type property`
`162`	`158`	`EditorGUI.PropertyField(propertyPosition, dataTypeProperty, new GUIContent("Type"), false);`
`163`	`159`	`propertyPosition.y += rawPropertyHeight;`
`164`		`- summaryFieldHeight += rawPropertyHeight;`
`165`	`160`
`166`	`161`	`//decide what property should be drawn depending on the folder data type`
`167`	`162`	`if (isPathBased)`
`168`	`163`	`{`
`169`	`164`	`propertyPosition.height = EditorGUI.GetPropertyHeight(pathProperty);`
`170`	`165`	`EditorGUI.PropertyField(propertyPosition, pathProperty, false);`
`171`	`166`	`propertyPosition.y += propertyPosition.height + Style.spacing;`
`172`		`- summaryFieldHeight += propertyPosition.height + Style.spacing;`
`173`	`167`	`propertyPosition.height = Style.height;`
`174`	`168`	`}`
`175`	`169`	`else`
`176`	`170`	`{`
`177`	`171`	`EditorGUI.PropertyField(propertyPosition, nameProperty, false);`
`178`	`172`	`propertyPosition.y += rawPropertyHeight;`
`179`		`- summaryFieldHeight += rawPropertyHeight;`
`180`	`173`	`}`
`181`	`174`
`182`	`175`	`propertyPosition.height = EditorGUI.GetPropertyHeight(tooltipProperty);`
`183`	`176`	`EditorGUI.PropertyField(propertyPosition, tooltipProperty, false);`
`184`	`177`	`propertyPosition.y += propertyPosition.height + Style.spacing;`
`185`		`- summaryFieldHeight += propertyPosition.height + Style.spacing;`
`186`	`178`	`propertyPosition.height = Style.height;`
`187`	`179`
`188`	`180`	`//adjust rect for folder icons + button strip`
`@@ -244,7 +236,6 @@ public override void OnGUI(Rect position, SerializedProperty property, GUIConten`
`244`	`236`	`EditorGUI.EndProperty();`
`245`	`237`	`}`
`246`	`238`
`247`		`-`
`248`	`239`	`private static class Style`
`249`	`240`	`{`
`250`	`241`	`internal static readonly float height = EditorGUIUtility.singleLineHeight;`
Original file line number	Diff line number	Diff line change
`@@ -29,15 +29,14 @@ protected virtual void DrawInvalidDrawerLabel(Rect position, string label)`
`29`	`29`	`EditorGUI.LabelField(position, content);`
`30`	`30`	`}`
`31`	`31`
`32`		`-`
`33`		`- public override sealed float GetPropertyHeight(MaterialProperty prop, string label, MaterialEditor editor)`
	`32`	`+ public sealed override float GetPropertyHeight(MaterialProperty prop, string label, MaterialEditor editor)`
`34`	`33`	`{`
`35`	`34`	`return IsPropertyValid(prop)`
`36`	`35`	`? GetPropertyHeightSafe(prop, label, editor)`
`37`	`36`	`: base.GetPropertyHeight(prop, label, editor);`
`38`	`37`	`}`
`39`	`38`
`40`		`- public override sealed void OnGUI(Rect position, MaterialProperty prop, string label, MaterialEditor editor)`
	`39`	`+ public sealed override void OnGUI(Rect position, MaterialProperty prop, string label, MaterialEditor editor)`
`41`	`40`	`{`
`42`	`41`	`if (IsPropertyValid(prop))`
`43`	`42`	`{`
Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,6 @@ public class MaterialCompactTextureDrawer : BaseMaterialPropertyDrawer`
`7`	`7`	`{`
`8`	`8`	`private readonly string tooltip;`
`9`	`9`
`10`		`-`
`11`	`10`	`public MaterialCompactTextureDrawer() : this(string.Empty)`
`12`	`11`	`{ }`
`13`	`12`
`@@ -16,7 +15,6 @@ public MaterialCompactTextureDrawer(string tooltip)`
`16`	`15`	`this.tooltip = tooltip;`
`17`	`16`	`}`
`18`	`17`
`19`		`-`
`20`	`18`	`protected override float GetPropertyHeightSafe(MaterialProperty prop, string label, MaterialEditor editor)`
`21`	`19`	`{`
`22`	`20`	`return EditorGUIUtility.singleLineHeight;`
Original file line number	Diff line number	Diff line change
`@@ -7,13 +7,11 @@ public abstract class MaterialConditionalDrawer : BaseMaterialPropertyDrawer`
`7`	`7`	`{`
`8`	`8`	`protected readonly string togglePropertyName;`
`9`	`9`
`10`		`-`
`11`	`10`	`protected MaterialConditionalDrawer(string togglePropertyName)`
`12`	`11`	`{`
`13`	`12`	`this.togglePropertyName = togglePropertyName;`
`14`	`13`	`}`
`15`	`14`
`16`		`-`
`17`	`15`	`protected override float GetPropertyHeightSafe(MaterialProperty prop, string label, MaterialEditor editor)`
`18`	`16`	`{`
`19`	`17`	`if (!HasToggle(prop))`
Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,6 @@ public class MaterialHelpDecorator : BaseMaterialPropertyDrawer`
`8`	`8`	`private readonly string message;`
`9`	`9`	`private readonly MessageType type;`
`10`	`10`
`11`		`-`
`12`	`11`	`public MaterialHelpDecorator(string message) : this(message, 1)`
`13`	`12`	`{`
`14`	`13`	`this.message = message;`
`@@ -20,7 +19,6 @@ public MaterialHelpDecorator(string message, float type)`
`20`	`19`	`this.type = (MessageType)type;`
`21`	`20`	`}`
`22`	`21`
`23`		`-`
`24`	`22`	`protected override float GetPropertyHeightSafe(MaterialProperty prop, string label, MaterialEditor editor)`
`25`	`23`	`{`
`26`	`24`	`var minHeight = 24.0f;`
`@@ -33,7 +31,6 @@ protected override void OnGUISafe(Rect position, MaterialProperty prop, string l`
`33`	`31`	`EditorGUI.HelpBox(position, message, type);`
`34`	`32`	`}`
`35`	`33`
`36`		`-`
`37`	`34`	`private static class Style`
`38`	`35`	`{`
`39`	`36`	`internal static readonly GUIStyle textStyle = new GUIStyle(EditorStyles.label)`
Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,6 @@ public class MaterialHideIfToggleDrawer : MaterialConditionalDrawer`
`5`	`5`	`public MaterialHideIfToggleDrawer(string togglePropertyName) : base(togglePropertyName)`
`6`	`6`	`{ }`
`7`	`7`
`8`		`-`
`9`	`8`	`protected override bool IsVisible(bool? value)`
`10`	`9`	`{`
`11`	`10`	`return value.HasValue && !value.Value;`
Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,6 @@ public class MaterialIndentDrawer : BaseMaterialPropertyDrawer`
`7`	`7`	`{`
`8`	`8`	`private readonly int indent;`
`9`	`9`
`10`		`-`
`11`	`10`	`public MaterialIndentDrawer() : this(1)`
`12`	`11`	`{ }`
`13`	`12`
`@@ -16,7 +15,6 @@ public MaterialIndentDrawer(float indent)`
`16`	`15`	`this.indent = (int)indent;`
`17`	`16`	`}`
`18`	`17`
`19`		`-`
`20`	`18`	`protected override float GetPropertyHeightSafe(MaterialProperty prop, string label, MaterialEditor editor)`
`21`	`19`	`{`
`22`	`20`	`return base.GetPropertyHeightSafe(prop, label, editor);`
Original file line number	Diff line number	Diff line change
`@@ -10,14 +10,12 @@ public class MaterialMinMaxSliderDrawer : BaseMaterialPropertyDrawer`
`10`	`10`	`private readonly float minValue;`
`11`	`11`	`private readonly float maxValue;`
`12`	`12`
`13`		`-`
`14`	`13`	`public MaterialMinMaxSliderDrawer(float minValue, float maxValue)`
`15`	`14`	`{`
`16`	`15`	`this.minValue = minValue;`
`17`	`16`	`this.maxValue = maxValue;`
`18`	`17`	`}`
`19`	`18`
`20`		`-`
`21`	`19`	`protected override float GetPropertyHeightSafe(MaterialProperty prop, string label, MaterialEditor editor)`
`22`	`20`	`{`
`23`	`21`	`return EditorGUIUtility.singleLineHeight;`
Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,6 @@ public class MaterialShowIfToggleDrawer : MaterialConditionalDrawer`
`5`	`5`	`public MaterialShowIfToggleDrawer(string togglePropertyName) : base(togglePropertyName)`
`6`	`6`	`{ }`
`7`	`7`
`8`		`-`
`9`	`8`	`protected override bool IsVisible(bool? value)`
`10`	`9`	`{`
`11`	`10`	`return value ?? false;`
Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,6 @@ public class MaterialTitleDecorator : BaseMaterialPropertyDrawer`
`8`	`8`	`private readonly GUIContent header;`
`9`	`9`	`private readonly float spacing;`
`10`	`10`
`11`		`-`
`12`	`11`	`public MaterialTitleDecorator(string header) : this(header, 4.0f)`
`13`	`12`	`{ }`
`14`	`13`
`@@ -18,7 +17,6 @@ public MaterialTitleDecorator(string header, float spacing)`
`18`	`17`	`this.spacing = spacing;`
`19`	`18`	`}`
`20`	`19`
`21`		`-`
`22`	`20`	`protected override float GetPropertyHeightSafe(MaterialProperty prop, string label, MaterialEditor editor)`
`23`	`21`	`{`
`24`	`22`	`return EditorGUIUtility.singleLineHeight + spacing;`