sdfx icon indicating copy to clipboard operation
sdfx copied to clipboard

Issues with widgetNames in UI

Open morgan55555 opened this issue 10 months ago • 1 comments

There is too much cloned defaults in widgetDefaults.ts, but it always will be not enough for endless custom nodes for ComfyUI. And if we change widgetNames to another (not from our node widget) - this will break correct widget loading in UI (tried with KSampler Config (rgthree), which has field "steps_total", if widgetNames is just "steps" - widget will show default value instead of real node value. If widgetNames is "steps_total" - there is no default/step/min/max values.)

Better solution will be using only widgetIdxs, remove all widgetNames and change it to widgetTypes (e.g. steps, cfg, etc...), which can be used for any selected widgetIdxs.

Also, sometimes may be need to set custom widget parameters instead of its type (min/max/step).

morgan55555 avatar Apr 25 '24 07:04 morgan55555