I think we should add Asset to the second message “Node group is not marked as tool” to guide users as to where to fix the problem.
E.g., “Asset not marked as tool”, or enabled as…
Approving as it is. Suggestion of future improvements:
Hi @Harley, what did you mean by:
Personally would rather discourage that by getting rid of the arbitrary sizes.
On a side note I talked to Julian and I think we should go…
@PratikPB2123 can you handle the commit + release notes? The code seems fine, so if you tested it, it is good to go.
Thanks @Andras-Borsanyi for the patch :)
This was discussed in a module meeting, however its benefits didn't seem to justify the changes.
@JulienKaspar shouldn't be the other way around? N is used universally in the editors, while T is supported only in a handful. For example all the animation editors use T for something else…
Indeed better to use min/max functions.
Also, is the jump height being preserved across the session? (similar to how we preserve the speed)
I actually committed this by mistake (29b67ad31b11). Let me know if you want me to revert it (and re-open it).