Comfy-Org/ComfyUI_frontend
1
0
Fork 0
mirror of https://github.com/Comfy-Org/ComfyUI_frontend.git synced 2026-02-18 22:10:03 +00:00
Code Issues Packages Projects Releases Wiki Activity

[BugFix] Workaround custom nodes expectation on DOMWidget.value (#2505)

Browse Source
This commit is contained in:
Chenlei Hu
2025-02-10 22:06:20 -05:00
committed by GitHub
parent d4122a7510
commit 1a06c91ed1
1 changed files with 12 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/scripts/domWidget.ts
View File

@@ -439,6 +439,18 @@ LGraphNode.prototype.addDOMWidget = function <
}
const widget = new DOMWidgetImpl(name, type, element, options)
// Workaround for https://github.com/Comfy-Org/ComfyUI_frontend/issues/2493
// Some custom nodes are explicitly expecting getter and setter of `value`
// property to be on instance instead of prototype.
Object.defineProperty(widget, 'value', {
get(this: DOMWidgetImpl<T, V>): V {
return this.options.getValue?.() ?? ('' as V)
},
set(this: DOMWidgetImpl<T, V>, v: V) {
this.options.setValue?.(v)
this.callback?.(this.value)
}
})
// Ensure selectOn exists before iteration
const selectEvents = options.selectOn ?? ['focus', 'click']