ComfyUI-Custom-Scripts icon indicating copy to clipboard operation
ComfyUI-Custom-Scripts copied to clipboard

Snap all group borders to grid, when [ Fit Group to Nodes ] is used and [ Always snap to grid ] is enabled.

Open JorgeR81 opened this issue 1 year ago • 0 comments

With this new commit, groups are snapped to grid after moving:

https://github.com/comfyanonymous/ComfyUI/pull/3393

But this works in a different way than the [ Fit Group to Nodes ] option.

When I use [ Fit Group to Nodes ], only the group's bottom border it's aligned with the grid ( not the top one ):

before_m

And after moving the group, it's the top border that's snapped to grid ( not the bottom one ), so the node positions are changed in relation to the group :

after2

after

So after moving, I still need to redimension the bottom border manually, to snap it to the grid:

fit

Once the group's top border it's aligned, the relative positions will always stay the same, after moving the group again. But every time I use [ Fit Group to Nodes ], the group's top border will be misaligned again. A solution would be to snap all borders to the grid, when [ Fit Group to Nodes ] is used, like on the last image.

JorgeR81 avatar May 27 '24 22:05 JorgeR81