Conditional UnityEvent inspector is broken

[nipunasudha]

My attempt to conditionally show/hide an UnityEvent field failed.

The inspector is broken like this

[Deadcows]

Yep, actually there is no way to draw custom property drawer inside of property drawer. So any property with fancy inspector appearance will be drawn as a plain data :(

Like with arrays, it's just the way unity works.

[Deadcows]

...buuut I found this thread on how to hack the engine to make it done, so maybe there is some hope :)

// I'll check it a bit later
https://forum.unity.com/threads/drawing-default-property-drawer-in-a-custom-property-drawer.197268/

[woistjadefox]

Works in my case. Is it already fixed?

[Deadcows]

Are you sure you checked on UnityEvent?
Tested it now and it looks like this:

	public bool A;
	[ConditionalField("A")] public UnityEvent B;

ConditionalField should handle most of the types now, but still struggles with such types like UnityEvent, which have Custom Drawer and base class

[woistjadefox]

Yes it's working:

I use Untiy 2019.3.0f3 maybe that's the reason why?

[woistjadefox]

Ahh.. sorry found out why. I use a custom property drawer for UnityEvents:
https://gist.github.com/ByronMayne/70a46e73f3af7fb9fec7437174dd4858

[Deadcows]

Whoa 😲. I never thought of possibility to extend existing custom drawers this way! Maybe I'll contact Bryon to embed this one in MyBox. Thanks for bringing this up 👍
Quite a few of interesting ideas he's got there btw

[Deadcows]

Finally! Now UnityEvent works with ConditionalAttribute and comes with a few nice improvements! Thanks to the Bryon, almost after half of a year I'm able to close this issue 😁

@woistjadefox, you probably should remove your CollapsableEventDrawer.cs before you'll grab the latest commit of MyBox