mcmonkeyprojects / sd-dynamic-thresholding

Dynamic Thresholding (CFG Scale Fix) for Stable Diffusion (eSwarmUI, ComfyUI, and Auto WebUI)
MIT License
1.1k stars 103 forks source link

Hiding controls when disabled is awkward and buggy #98

Closed heftig closed 4 months ago

heftig commented 5 months ago

I think the extension should not hide its controls while it's disabled. This makes it really awkward to use:

At the very least, the controls should not be hidden; that's what the accordion is for.

Ideally, I would like to see it work like the Hires. fix and the Refiner accordions, where opening and closing the accordion automatically enables and disables the extension, except if the user has clicked the checkbox before.

heftig commented 5 months ago

Looks like the visibility gets fixed by https://github.com/mcmonkeyprojects/sd-dynamic-thresholding/pull/97.