Differential D10241
Nodes: Move the "Group Input" and "Group Output" nodes to consistent menu location Authored by Hans Goudey (HooglyBoogly) on Jan 28 2021, 10:25 PM. Tags Subscribers None
Details  Currently (in geometry nodes) you can delete the group input or group output nodes with no Additionally, these nodes are moved to the "Group" submenu for all node editors.
Diff Detail
Event TimelineComment Actions I wonder if the categories are correct. Those are not Layout items. I would either have them in Group or in Input/Output categories. For the records, for shaders the Material Output is in the output category. Also another option is for the poll function to only let you create the nodes if there is no node of its type there. Comment Actions I agree with Dalai here that they should be under Input and Output respectively. Currently, having more than one Group Output node disables the whole tree. Could it be made to only use the active Group Output node, like Material Output does? Comment Actions To be clear, we have no "Output" category in geometry nodes right now, are you suggesting adding one?
I think this situation is a bit different, since these are just node groups. We could add the idea of an active node group output, but as far as I know we had some different designs for how node groups could be improved already. Comment Actions I talked to @Hans Goudey (HooglyBoogly) and we both think it could be in Group. @Pablo Vazquez (pablovazquez) do you have any big objection with that? I personally find a bit strange to have the Output category that we won't really be using much Comment Actions Let's go with adding these into the Group category. These nodes should be moved from Input/Output to Group for Shading, Texture, and Compositor nodes as well.  Comment Actions
|