Support non-public fields too

[XtroTheArctic]

This may not be considered as a bug but I think it is because a serialized field in Unity is a serialized field, no matter if it's public or not.

My icon field is private and serialized but it doesn't display the icon in project view.

[SerializeField] [AssetIcon] Sprite Icon;

To support private serialized Unity fields, GetMembers call in AssetDrawerFactoryLibrary.cs should be changed to this:

var typeMembers = type.GetMembers(BindingFlags.NonPublic | BindingFlags.Public | BindingFlags.Instance);

[XtroTheArctic]

Scratch the modified line in my original comment.

We need this entire block to be able to support inherited private fields.

                    var typeMembers = new List<MemberInfo>();
                    var typeToGetMembers = type;

                    do
                    {
                        typeMembers.AddRange(typeToGetMembers.GetMembers(BindingFlags.NonPublic | BindingFlags.Public | BindingFlags.Instance | BindingFlags.DeclaredOnly));

                        typeToGetMembers = typeToGetMembers.BaseType;
                    }
                    while (typeToGetMembers != null);

[XtroTheArctic]

@fydar would you like to add this support please?

[XtroTheArctic]

Hello. Is this project dead? It would be sad to see such a nice tool go out of active development/support. :(

[ChuckinAllen]

If your looking to display the icon without having to make it public, I found a simple work around that you could use so you don't have the sprite exactly public you still have to have at least one field public but you could have 2 fields one public and the other private or sterilized, then just assign the public filed with the private one with OnValidate, might not be the most proficient but if you don't what to allow access to the other field this would work. You could also use the [HideInInspector] attribute on the public field so that you only display the one field.
Here is the example of what I did:

using AssetIcons;
using UnityEngine;

[CreateAssetMenu(fileName = "TEST", menuName = "TEST")]
public class IconTester : ScriptableObject
{
    [SerializeField] private Sprite Sprite;

    [AssetIcon]
    [HideInInspector]
    public Sprite EditorScriptableObjectDisplay;

    private void OnValidate()
    {
        if (Sprite != null && EditorScriptableObjectDisplay == null)
        {
            EditorScriptableObjectDisplay = Sprite;
        }
        if(Sprite == null)
        {
            EditorScriptableObjectDisplay = null;
        }
    }
}

[XtroTheArctic]

Thank you for you suggestion. An interesting one. I'm still using my modification to the original script I shared above.

It's a shame that this project got abandoned thou. I hope the author will return some day.

[ChuckinAllen]

Ya it is it's a really cool project, also found this code works a bit better, with less calls.

using AssetIcons;
using UnityEngine;

[CreateAssetMenu(fileName = "TEST", menuName = "TEST")]
public class IconTester : ScriptableObject
{
    [SerializeField] private Sprite Sprite;

    [AssetIcon]
    [HideInInspector]
    public Sprite EditorScriptableObjectDisplay;

    private void OnValidate()
    {
        if(Sprite != EditorScriptableObjectDisplay)
        {
            EditorScriptableObjectDisplay = Sprite;
        }
    }
}

[fydar]

I really gotta make a 3.0 of this asset; I've become a much better programmer since 😳

[ChuckinAllen]

Defiantly should, it's a really cool asset and works really seamlessly for the most part! Also glad I said something and thank you for the response! ❤️