Fix AssetPreview height for targets without the preview texture

arimger · arimger · commit 34baad5553c8 · 2022-05-07T23:32:43.000+02:00
diff --git a/Assets/Editor Toolbox/Editor/Drawers/Regular/AssetPreviewAttributeDrawer.cs b/Assets/Editor Toolbox/Editor/Drawers/Regular/AssetPreviewAttributeDrawer.cs
@@ -56,8 +56,9 @@ private void DrawAssetPreview(Rect rect, Texture2D previewTexture)
 
         protected override float GetPropertyHeightSafe(SerializedProperty property, GUIContent label)
         {
-            //return native height 
-            if (!property.objectReferenceValue)
+            var target = property.objectReferenceValue;
+            var previewTexture = AssetPreview.GetAssetPreview(target);
+            if (previewTexture == null)
             {
                 return base.GetPropertyHeightSafe(property, label);
             }

Original file line number	Diff line number	Diff line change
`@@ -56,8 +56,9 @@ private void DrawAssetPreview(Rect rect, Texture2D previewTexture)`
`56`	`56`
`57`	`57`	`protected override float GetPropertyHeightSafe(SerializedProperty property, GUIContent label)`
`58`	`58`	`{`
`59`		`- //return native height`
`60`		`- if (!property.objectReferenceValue)`
	`59`	`+ var target = property.objectReferenceValue;`
	`60`	`+ var previewTexture = AssetPreview.GetAssetPreview(target);`
	`61`	`+ if (previewTexture == null)`
`61`	`62`	`{`
`62`	`63`	`return base.GetPropertyHeightSafe(property, label);`
`63`	`64`	`}`