Skip to content

fix: normalize fast groups muter widget size#735

Open
fengwk wants to merge 1 commit into
rgthree:mainfrom
fengwk:dev
Open

fix: normalize fast groups muter widget size#735
fengwk wants to merge 1 commit into
rgthree:mainfrom
fengwk:dev

Conversation

@fengwk

@fengwk fengwk commented Jun 15, 2026

Copy link
Copy Markdown

Summary

This fixes a transient sizing issue in Fast Groups Muter where the widget row width can be inconsistent after refresh / load, causing the first row to render with an incorrect length.

Tested environment

  • ComfyUI core: v0.24.0
  • ComfyUI commit: f49bdb65
  • comfyui-frontend-package: 1.44.19

Validated on the classic / non-Vue node path.

Problem

Fast Groups Muter uses a temporary size (tempSize) while widgets are being refreshed.

After that temporary sizing window ends, the node can keep a stale width for one render phase, which makes widget row drawing inconsistent even though the final computed size should be smaller / normalized.

Changes

  • add a follow-up normalization step after the temporary sizing debounce clears
  • recompute the final node size with super.computeSize(...)
  • call setSize(...) if the current node size no longer matches the normalized size
  • mark the canvas dirty so the corrected layout is redrawn immediately

Why this helps

This preserves the existing temporary sizing behavior during refresh, but ensures the node settles back to its real computed size once the refresh is complete.

Manual validation

Validated manually with a workflow containing Fast Groups Muter where the first widget row previously rendered with inconsistent width/length.
After this change, widget row widths are consistent after load/refresh.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant