From e139816db0d18e58d299ad1f61401ffcf0bcb3a8 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Tue, 3 Feb 2026 23:09:29 +0000 Subject: [PATCH] Fix Efficient Loader UI layout issue with font size changes Co-authored-by: jags111 <5968619+jags111@users.noreply.github.com> --- js/widgethider.js | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/js/widgethider.js b/js/widgethider.js index a270872..afba9fe 100644 --- a/js/widgethider.js +++ b/js/widgethider.js @@ -36,7 +36,6 @@ function toggleWidget(node, widget, show = false, suffix = "") { node.setSize([node.size[0], newHeight]); } -const WIDGET_HEIGHT = 24; // Use for Multiline Widget Nodes (aka Efficient Loaders) function toggleWidget_2(node, widget, show = false, suffix = "") { if (!widget || doesInputWithNameExist(node, widget.name)) return; @@ -52,8 +51,10 @@ function toggleWidget_2(node, widget, show = false, suffix = "") { widget.computeSize = show ? origProps[widget.name].origComputeSize : () => [0, -4]; if (initialized){ - const adjustment = show ? WIDGET_HEIGHT : -WIDGET_HEIGHT; - node.setSize([node.size[0], node.size[1] + adjustment]); + // Calculate the new height for the node based on its computeSize method + // This properly accounts for font size changes and ensures widgets don't overlap + const newHeight = node.computeSize()[1]; + node.setSize([node.size[0], newHeight]); } }