Conditional UnityEvent inspector is broken about mybox HOT 8 CLOSED

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

from mybox.

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.

from mybox.

...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/

from mybox.

Works in my case. Is it already fixed?

from mybox.

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

from mybox.

Yes it's working:

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

from mybox.

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

from mybox.

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

from mybox.