Some UI feedback on version 2.9

[timv]

Hi Gavalakis,

I have been testing the 2.9 version of NodeCanvas and have a few points of feedback:

The new dark boxes on Action nodes decrease the contrast with the text, which is unfortunate. I think I prefer the older version. Also, what is the reason for the added dot in front of all tasks? No dots feels cleaner to me.

In my project sequential task nodes are by far the most common. Would you consider removing the (In Sequence), and only showing (In Parallel) instead? Before this change I actually added the word in all caps “PARALLEL” to action nodes that were configured like this. I think this would reduce the amount of redundant information on screen. I think I’d prefer that, but as you said before, it’s impossible to please everyone.

It seems icons no longer show in the Node browser window (“Add BehaviourTree Node”). Were these removed on purpose?

Thanks for lowering the minimum search string length to 2. 🙂

[Gavalakis]

Hello again 🙂

— I am on the Unity dark theme 99% of the time when working with Unity and NC. I do agree that the old (non-darkbox) looks better in the light theme, but the new dark-box one looks better in the dark theme. I try to avoid “if (isProSkin)” as much as possible, but for this case I think I will remove the darkbox from the light theme in the next version. You can do this now, in Styles.cs line #55:

1 2 3 4 5 6 7 8 9 10 11 12

private static GUIStyle _roundedBox; public static GUIStyle roundedBox{ get { if (UnityEditor.EditorGUIUtility.isProSkin){ return _roundedBox?? (_roundedBox = new GUIStyle( (GUIStyle)"ShurikenEffectBg") ); } return GUIStyle.none; } }

— The dots were added (hopefully) for easier distrinquish between actions in the list. Maybe I remove them. I will let it roll for a bit more and see if it actually does help (or not).

— Labeling “InSequencer” and “InParallel” explicitely was actually a request from another guy (whom I recall was working in FSMs) 🙂 It does makes more sense in the FSMs context for both to exist. I will need to think about before changing this again.

Thanks!

Join us on Discord: https://discord.gg/97q2Rjh

[timv]

Thanks for your replies. I’ll try out your suggestion, thanks!

[Author]

Posts