Reply To: Suggestions

[Gavalakis]

Hey,

Thanks a lot for the suggestions! Let me address them in order:

1) DELETE key should delete the selected connection. I’ve just re-checked to confirm and works as expected. Does this always no work for you, or sometimes?

2) Sure, I will add ctrl c/v shortcuts for copy/paste.
Initially I didn’t exactly because copy/paste is available in mouse menu :).

3) Yeah. Default values for ports is something I wanted to add for some time now. I will try to get them in the next version!
For normal nodes (ones extending FlowNode/FlowControlNode etc), I am leaning towards something like this:

1 2 3

AddValueInput<float>("number", 13f);

By the way it’s already possible to do this (a bit hacky though, but better than writing an inspector just for this 🙂 ):

1 2 3

AddValueInput<float>("number").serializedValue = 13f;

For Simplex nodes, indeed a [DefaultValue] attribute is the best option to add and I will do.

4) Enter does work for adding the highlighted node, but if the search field is still in focus, the first Enter hit will be consumed by the search field. The second one will work. If search field is not in focus 1 Enter hit should work fine.
I haven’t yet found a good solution around this, but can you confirm that it does even if with the above constrains?

5) Sure. I am thinking of adding an enum parameter to the existing [Icon] attribute to specify if the icon is Solo, or WithName for example. So this could be used like this [Icon(“Keyboard”, IconAttribute.Mode.WithName )]. Just an example.

6) It’s true that a string can’t be set back to null right now. The underline problem is quite similar to how a string can’t be null if it’s exposed in a Unity inspector (with a public string field in a monobehaviour for example).
I will investigate this. I am thinking that I could add a small button next to null-able type fields with witch to set them to their default value (which of course for string is null).

7) There is no way to do this, and unfortunately that will require some heavy code changes and special case ifs, to allow it and I dont think that adding this many changes will really worth it just for some very special cases like this one 🙁
To be honest I also think that doing this, will result is some big UI confusions.
Unfortunately I don’t see this option coming, at least any time soon. I am really sorry about that!

Finally, I want to say thank you for all your suggestions as well as your positive feedback!
I just now realized you are from the Punch Club dev team. Punch Club was awesome! I spend a lot of hours playing it! 😉
Graveyard Keeper looks great as well. Looking forward to it!
Do you mind if I add Graveyard Keeper in the new showcase page when it’s ready? (will be similar to NodeCanvas showcase page)

Thanks!

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