TinyTerra / ComfyUI_tinyterraNodes

A selection of nodes for Stable Diffusion ComfyUI
GNU General Public License v3.0
421 stars 45 forks source link

Glow for link in "straight style" mode. #13

Closed pvp-by closed 6 months ago

pvp-by commented 1 year ago

Thank you so much for your good work. I really like the realization of the Link Style (ttN). But in " straight style " mode all links tend to overlap and turn into 1 common link. and when you select a node the link highlighting goes only up to the common line example : Figure_7

maybe in this "style mode straight" you can add more glow effect so that even a link that is at the bottom of the layers, still gives a hint where it goes. example : Figure_8

TinyTerra commented 1 year ago

I agree that the straight style isn't the clearest when it comes to overlapping lines, however the current implementation just adds a UI option within the comfy interface for an inbuilt function of LiteGraph... I'm not sure at the moment how litegraph handles line overlay priority, but I'll have a look into it

EricRollei commented 1 year ago

Hi, thanks for your nodes! I'm just adding onto this thread here - it seems that your link style duplicates or conflicts with the Pythongosssss custom scripts https://github.com/pythongosssss/ComfyUI-Custom-Scripts I was trying to save the spline curve as default but it kept going to straight and I think that is your default. I prefer to have the spline curve node links so if there's a way to either deactivate that one feature or set the default to curve that would be great. I think I can set the enable interface to False, but then I'd loose some functionality that I like.

TinyTerra commented 1 year ago

If you right click on a link and change the ttn link style it will set the default for tinyterra - if that doesn't persist then I'll need to do some debugging. Let me know