alphard/ComfyUI-Lora-Manager
1
0
Fork 0
mirror of https://github.com/willmiao/ComfyUI-Lora-Manager.git synced 2026-03-21 21:22:11 -03:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #581 from willmiao/codex/fix-typeerror-in-autocomplete.js

Browse Source 
fix: clean up autocomplete event handlers
This commit is contained in:
pixelpaws
2025-10-21 19:33:02 +08:00
committed by GitHub
parent 46126f9950 e043537241
commit e63ef8d031
1 changed files with 57 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

73
web/comfyui/autocomplete.js
View File

@@ -156,10 +156,15 @@ class AutoComplete {
this.currentSearchTerm = '';
this.previewTooltip = null;
this.previewTooltipPromise = null;
// Initialize TextAreaCaretHelper
this.helper = new TextAreaCaretHelper(inputElement, () => app.canvas.ds.scale);
this.onInput = null;
this.onKeyDown = null;
this.onBlur = null;
this.onDocumentClick = null;
this.init();
}
@@ -233,29 +238,42 @@ class AutoComplete {
bindEvents() {
// Handle input changes
this.inputElement.addEventListener('input', (e) => {
this.onInput = (e) => {
this.handleInput(e.target.value);
});
};
this.inputElement.addEventListener('input', this.onInput);
// Handle keyboard navigation
this.inputElement.addEventListener('keydown', (e) => {
this.onKeyDown = (e) => {
this.handleKeyDown(e);
});
};
this.inputElement.addEventListener('keydown', this.onKeyDown);
// Handle focus out to hide dropdown
this.inputElement.addEventListener('blur', (e) => {
this.onBlur = () => {
// Delay hiding to allow for clicks on dropdown items
setTimeout(() => {
this.hide();
}, 150);
});
};
this.inputElement.addEventListener('blur', this.onBlur);
// Handle clicks outside to hide dropdown
document.addEventListener('click', (e) => {
if (!this.dropdown.contains(e.target) && e.target !== this.inputElement) {
this.onDocumentClick = (e) => {
if (!this.dropdown) {
return;
}
const target = e.target;
if (!(target instanceof Node)) {
return;
}
if (!this.dropdown.contains(target) && target !== this.inputElement) {
this.hide();
}
});
};
document.addEventListener('click', this.onDocumentClick);
}
handleInput(value = '') {
@@ -450,6 +468,10 @@ class AutoComplete {
}
hide() {
if (!this.dropdown) {
return;
}
this.dropdown.style.display = 'none';
this.isVisible = false;
this.selectedIndex = -1;
@@ -579,7 +601,27 @@ class AutoComplete {
if (this.debounceTimer) {
clearTimeout(this.debounceTimer);
}
if (this.onInput) {
this.inputElement.removeEventListener('input', this.onInput);
this.onInput = null;
}
if (this.onKeyDown) {
this.inputElement.removeEventListener('keydown', this.onKeyDown);
this.onKeyDown = null;
}
if (this.onBlur) {
this.inputElement.removeEventListener('blur', this.onBlur);
this.onBlur = null;
}
if (this.onDocumentClick) {
document.removeEventListener('click', this.onDocumentClick);
this.onDocumentClick = null;
}
if (typeof this.behavior.destroy === 'function') {
this.behavior.destroy(this);
} else if (this.previewTooltip) {
@@ -593,7 +635,6 @@ class AutoComplete {
this.dropdown = null;
}
// Remove event listeners would be added here if we tracked them
}
}