search

comfyanonymous / ComfyUI

The most powerful and modular diffusion model GUI, api and backend with a graph/nodes interface.
https://www.comfy.org/
GNU General Public License v3.0
54.63k stars 5.76k forks source link

[Feature Request] Make groups stick to the grid... and allow locking groups too #566

Open DivinoAG opened 1 year ago

DivinoAG commented 1 year ago

Per title. It would be great if node groups would also snap to grid when moving or resizing, just like nodes do now, that way we don't move every node out of the grid if we move the group anywhere.

While we're at it, please add a feature to lock groups in place so that they (or their child nodes) can't be moved unless explicitly unlocked back. I have wasted way too much time fixing an entire workflow because I accidentally moved a whole group while trying to pan the view, making it land over other nodes which now get moved as well if I try to move that group back to its place.

michaelLochlann commented 11 months ago

Also looking for this. Groups should be able to be snapped to the grid. Thanks :)

Woisek commented 11 months ago

I may add, that an undo function would also be a great help against an accidentally moving of nodes.

tuomashatakka commented 6 months ago

I actually did this already on my local copy - I'll open a PR later today! I mean the snapping being on by default.

The group background nodes don't snap to grid at all - I wonder if that should be a separate pull request. What do you think?