Comfy-Org / Comfy-Org/ComfyUI_frontend

vueNodes: hover does not work while the simplified (zoomed-out) mode is active

Open
#15,316 1 comment 1 reaction 1 assignee Claimed by @benjcooley View on GitHub
area:litegraph area:ui area:vue-migration Potential Bug
Dominant language
TypeScript
Stars
2k
Forks
699
Avg merge
1d 7h
Merged PRs (30d)
490

Description

## Summary

`LiteGraph.vueNodesSuspended` restores clicking while Vue nodes are unmounted below the LOD threshold, but not hover. `processMouseMove` still takes the Vue-nodes branch, so `node_over` never resolves and nothing highlights under the cursor at low zoom.

## Detail

`vueNodesSuspended` has one read site (`LGraphCanvas.ts:2779`), which fans into a local `suspended` consulted at six branch guards inside the click path. The mouse-move path is not one of them: it still gates on `vueNodesMode` alone, so while suspended the canvas neither hovers a Vue node (there is none) nor falls back to litegraph hover.

User-visible effect: zoomed out, boxes give no hover feedback, so there is no affordance telling you a box is interactive before you click it. Clicking does work.

## Caution for whoever fixes this

Do not simply add the suspension term to the mouse-move gate. That path reaches `getWidgetOnPos` (around `LGraphCanvas.ts:3412`), and re-enabling widget hit-testing at a zoom where a whole node is roughly 20px was the original defect behind the click-path work. Hover highlight and widget hit-testing need separating first.

## Affected area

- `src/lib/litegraph/src/LGraphCanvas.ts`

Found during review of #15031.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.