Order not respected

[RDeluxe]

Hey there

I have been having some troubles with DeComment not respecting the order.

Here is the code :

        [DeHeader("Generic skills properties", mode = DeHeaderAttribute.Mode.BottomDivider)]
        [DeComment("Launch passively, disable the button to launch in UI")]
        [DeDivider]
        public bool Passive;

If I revert them everything is fine:

        [DeDivider]
        [DeComment("Launch passively, disable the button to launch in UI")]
        [DeHeader("Generic skills properties", mode = DeHeaderAttribute.Mode.BottomDivider)]
        public bool Passive;

Note: using only DeComment and DeHeader, order is screwed too. Same thing with DeComment + DeDivider

        [DeHeader("Generic skills properties", mode = DeHeaderAttribute.Mode.BottomDivider)]
        [DeComment("Launch passively, disable the button to launch in UI")]
        public bool Passive;

[Demigiant]

That is weird :O Checking it out!

[Demigiant]

Uh, it gets weirder. Everything works perfectly here, and order is respected (though I'm testing on Unity 5.6.3). Also, I checked the code and the order is decided by Unity, which should just call GetHeight and OnGUI of those drawers in the order they're drawn, so it seems Unity is messing something up there. On what version are you?

[RDeluxe]

Hm, using 2017.2 with .NET 4.6

[Demigiant]

In theory there is an extra order parameter on all attributes that you can use, to force the draw order:
[DeComment("dum dee dum very useful comment", order = 1)]

I'm assuming that in pre-2017 versions Unity ordered stuff correctly even if you left it to the default 0, while in 2017 order is now broken and you should set it manually :( Can you try and let me know (I don't want to make a 2017 copy of the DemiLib project just for this, if I can avoid it :P)

[Demigiant]

P.S. even so, I hope this is just a 2017.2 issue (since 2017.2 brought a lot of bugs) and it will be fixed in the next one?

[RDeluxe]

Thanks, it's working great with the order parameter !

Let's hope it's 2017.2 !

[Demigiant]

Crossing my fingers!