nodes/PozzettiAndrea-ComfyUI-GeometryPack
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Batch + Multi bars: remove Mode selector

Browse Source 
Mode is controlled solely by each node's own mode input widget, matching the
Dual and VTK preview policy. Bars keep navigation/layout controls + fullscreen.
This commit is contained in:
PozzettiAndrea
2026-08-14 15:32:32 +02:00
parent 7f927324b6
commit 15e6ffa02a
2 changed files with 3 additions and 56 deletions
Download Patch File Download Diff File Expand all files Collapse all files
javascript/js/mesh_preview_multi.js
+1 -26
View File
@@ -82,18 +82,9 @@ app.registerExtension({
gridControls.appendChild(Object.assign(document.createElement("span"), { textContent: "Rows:" }));
gridControls.appendChild(rowsInput);
// Mode selector (fields | texture) mirroring the node's `mode`
// input. NOT client-side: mode changes what gets EXPORTED, so
// changing it queues a re-run (same as editing the node widget).
const modeSel = document.createElement("select");
modeSel.style.cssText = "background:#333;color:#ccc;border:1px solid #555;border-radius:3px;font:11px monospace;padding:2px 6px;";
modeSel.innerHTML = '<option value="fields">Fields</option><option value="texture">Texture</option>';
modeSel.title = "Visualization mode (re-runs the node)";
// (Mode is controlled solely by the node's `mode` input widget.)
controls.appendChild(Object.assign(document.createElement("span"), { textContent: "Layout:" }));
controls.appendChild(layoutSel);
controls.appendChild(Object.assign(document.createElement("span"), { textContent: "Mode:" }));
controls.appendChild(modeSel);
controls.appendChild(gridControls);
bar.appendChild(collapseBtn);
bar.appendChild(controls);
@@ -216,22 +207,6 @@ app.registerExtension({
colsInput.addEventListener('change', onDim("cols"));
rowsInput.addEventListener('change', onDim("rows"));
// Mode: bar select <-> node widget, two-way (bar change re-runs).
const modeWidget = this.widgets?.find(w => w.name === "mode");
if (modeWidget) {
modeSel.value = modeWidget.value || "fields";
const origModeCb = modeWidget.callback;
modeWidget.callback = function(value) {
const res = origModeCb?.apply(this, arguments);
modeSel.value = value;
return res;
};
}
modeSel.addEventListener("change", () => {
if (modeWidget) modeWidget.value = modeSel.value;
app.queuePrompt();
});
syncLayoutUI();
applyCollapsed();
javascript/js/mesh_preview_vtk_batch.js
+2 -30
View File
@@ -99,29 +99,19 @@ app.registerExtension({
"border-radius:3px;font:11px monospace;padding:2px 6px;cursor:pointer;";
dropdown.innerHTML = '<option value="0">1 / 1</option>';
// Assemble navigation bar
// Mode selector (fields | texture) mirroring the node's `mode`
// input. NOT client-side: mode changes what gets EXPORTED (VTP/STL
// vs GLB), so changing it queues a re-run -- exactly like editing
// the node widget, just reachable from the nav bar.
const modeSel = document.createElement("select");
modeSel.style.cssText = "background:#333;color:#ccc;border:1px solid #555;" +
"border-radius:3px;font:11px monospace;padding:2px 6px;cursor:pointer;";
modeSel.innerHTML = '<option value="fields">Fields</option><option value="texture">Texture</option>';
modeSel.title = "Visualization mode (re-runs the node)";
// Fullscreen toggle -- expands the whole widget (viewer + this nav
// bar) so the arrows and dropdown stay usable in fullscreen.
// (Mode is controlled solely by the node's `mode` input widget.)
const fsButton = document.createElement("button");
fsButton.textContent = "⛶"; // fullscreen glyph
fsButton.title = "Fullscreen (← → to navigate)";
fsButton.style.cssText = "padding:4px 10px;cursor:pointer;background:#333;color:#ccc;border:1px solid #555;border-radius:3px;font-size:13px;line-height:1;";
// Assemble navigation bar
navBar.appendChild(prevButton);
navBar.appendChild(indexLabel);
navBar.appendChild(dropdown);
navBar.appendChild(nextButton);
navBar.appendChild(modeSel);
navBar.appendChild(fsButton);
// Create mesh info panel
@@ -207,23 +197,6 @@ app.registerExtension({
nextButton.addEventListener("click", () => showIndex(currentIndex + 1));
dropdown.addEventListener("change", () => showIndex(Number(dropdown.value)));
// Mode: bar select <-> node widget, two-way. Bar change re-runs
// (new exports needed); widget edits keep the bar in sync.
const modeWidget = this.widgets?.find(w => w.name === "mode");
if (modeWidget) {
modeSel.value = modeWidget.value || "fields";
const origModeCb = modeWidget.callback;
modeWidget.callback = function(value) {
const res = origModeCb?.apply(this, arguments);
modeSel.value = value;
return res;
};
}
modeSel.addEventListener("change", () => {
if (modeWidget) modeWidget.value = modeSel.value;
app.queuePrompt();
});
// Fullscreen the whole widget. The arrow BUTTONS stay clickable in
// fullscreen; arrow KEYS also navigate while the widget (nav bar) has
// focus. All listeners are on our own `container`, never document, so
@@ -439,7 +412,6 @@ app.registerExtension({
currentBatchSize = message.batch_size?.[0] || files.length;
if (indexWidget) indexWidget.options.max = currentBatchSize - 1;
modeSel.value = batchData.mode; // reflect the mode actually run
// (Re)populate the dropdown: one option per mesh, labelled by its
// source name (e.g. "apple.ply"), falling back to its position.