nodes/Acly-comfyui-tooling-nodes
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Parameter node: keep configured default values when reloading #25

Browse Source 
- make sure default is changed if the node is reconnected to a non-matching type
This commit is contained in:
Acly
2024-10-29 12:13:42 +01:00
parent e86973fedf
commit 035c68c629
1 changed files with 16 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
js/external_tooling_nodes.js
+16 -2
View File
@@ -83,6 +83,17 @@ function defaultParameterType(widgetType, connectedNode, connectedWidget) {
return paramType
}
function valueMatchesType(value, type, options) {
if (type === "number") {
return typeof value === "number"
} else if (type === "combo") {
return options?.values?.includes(value)
} else if (type === "toggle") {
return typeof value === "boolean"
}
return typeof value === "string"
}
function changeWidget(widget, type, value, options) {
widget.type = type
widget.value = value
@@ -99,8 +110,11 @@ function changeWidgets(node, type, connectedNode, connectedWidget) {
node.widgets[1].value = defaultParameterType(type, connectedNode, connectedWidget)
node.widgets[1].options = {values: parameterTypes[type]}
}
if (node.widgets[2].type !== type) {
changeWidget(node.widgets[2], type, connectedWidget.value, options)
const oldDefault = node.widgets[2].value
const isDefaultValid = valueMatchesType(oldDefault, type, connectedWidget.options)
if (node.widgets[2].type !== type || !isDefaultValid) {
const value = isDefaultValid && oldDefault !== "" ? oldDefault : connectedWidget.value
changeWidget(node.widgets[2], type, value, options)
}
if (type === "number" && node.widgets[3].value === 0 && node.widgets[4].value === 0) {
changeWidget(node.widgets[3], "number", options?.min ?? 0, options)