Skip to content

[backport core/1.41] Restore hiding of linked inputs in app mode#9676

Merged
AustinMroz merged 1 commit intocore/1.41from
backport-9671-to-core-1.41
Mar 9, 2026
Merged

[backport core/1.41] Restore hiding of linked inputs in app mode#9676
AustinMroz merged 1 commit intocore/1.41from
backport-9671-to-core-1.41

Conversation

@comfy-pr-bot
Copy link
Member

@comfy-pr-bot comfy-pr-bot commented Mar 9, 2026

Backport of #9671 to core/1.41

Automatically created by backport workflow.

┆Issue is synchronized with this Notion page by Unito

As a temporary fix for widgets being incorrectly hidden, #9669 allowed
all disabled widgets to be displayed.

This PR provides a more robust implementation to derive whether the
widget, as would be displayed from the root graph, is disabled.

Potential regression:
- Drag drop handlers are applied on node, not widgets. A subgraph
containing a "Load Image" node, does not allow dragging and dropping an
image onto the subgraphNode in order to load it. Because app mode
widgets would display from the original owning node prior to this PR,
these drag/drop handlers would apply. Placing "Load Image" nodes. I
believe this change makes behavior more consistent, but it warrants
consideration.

┆Issue is synchronized with this [Notion
page](https://www.notion.so/PR-9671-Restore-hiding-of-linked-inputs-in-app-mode-31e6d73d365081688e37fbb931f3af68)
by [Unito](https://www.unito.io)
@comfy-pr-bot comfy-pr-bot requested a review from a team as a code owner March 9, 2026 20:26
@comfy-pr-bot comfy-pr-bot added the backport Backporting a PR onto a release candidate label Mar 9, 2026
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Mar 9, 2026
@github-actions
Copy link

github-actions bot commented Mar 9, 2026

🎭 Playwright: ✅ 556 passed, 0 failed · 2 flaky

📊 Browser Reports
  • chromium: View Report (✅ 543 / ❌ 0 / ⚠️ 2 / ⏭️ 10)
  • chromium-2x: View Report (✅ 2 / ❌ 0 / ⚠️ 0 / ⏭️ 0)
  • chromium-0.5x: View Report (✅ 1 / ❌ 0 / ⚠️ 0 / ⏭️ 0)
  • mobile-chrome: View Report (✅ 10 / ❌ 0 / ⚠️ 0 / ⏭️ 0)

@github-actions
Copy link

github-actions bot commented Mar 9, 2026

🎨 Storybook: ✅ Built — View Storybook

Details

⏰ Completed at: 03/09/2026, 08:28:03 PM UTC

Links

@github-actions
Copy link

github-actions bot commented Mar 9, 2026

⚡ Performance Report

ℹ️ Collecting baseline variance data (0/5 runs). Significance will appear after 2 main branch runs.

Metric Baseline PR Δ
canvas-idle: style recalcs 124 124 -0%
canvas-idle: layouts 0 0 +0%
canvas-idle: task duration 420ms 396ms -6%
canvas-mouse-sweep: style recalcs 181 182 +1%
canvas-mouse-sweep: layouts 12 12 +0%
canvas-mouse-sweep: task duration 1001ms 1015ms +1%
dom-widget-clipping: style recalcs 43 44 +2%
dom-widget-clipping: layouts 0 0 +0%
dom-widget-clipping: task duration 363ms 368ms +1%
subgraph-dom-widget-clipping: style recalcs 73 75 +2%
subgraph-dom-widget-clipping: layouts 0 0
subgraph-dom-widget-clipping: task duration 427ms 420ms -2%
subgraph-idle: style recalcs 121 121 +0%
subgraph-idle: layouts 0 0 +0%
subgraph-idle: task duration 381ms 368ms -3%
subgraph-mouse-sweep: style recalcs 161 162 +1%
subgraph-mouse-sweep: layouts 16 16 +0%
subgraph-mouse-sweep: task duration 806ms 851ms +6%
Raw data
{
  "timestamp": "2026-03-09T20:30:57.581Z",
  "gitSha": "92b362db9492bef3dd8710b251f14908bd56b25f",
  "branch": "backport-9671-to-core-1.41",
  "measurements": [
    {
      "name": "canvas-idle",
      "durationMs": 2039.7629999999936,
      "styleRecalcs": 124,
      "styleRecalcDurationMs": 18.693,
      "layouts": 0,
      "layoutDurationMs": 0,
      "taskDurationMs": 423.556,
      "heapDeltaBytes": -3819748
    },
    {
      "name": "canvas-idle",
      "durationMs": 2029.2719999999917,
      "styleRecalcs": 124,
      "styleRecalcDurationMs": 19.636,
      "layouts": 0,
      "layoutDurationMs": 0,
      "taskDurationMs": 381.08699999999993,
      "heapDeltaBytes": -4192684
    },
    {
      "name": "canvas-idle",
      "durationMs": 2012.6010000000178,
      "styleRecalcs": 123,
      "styleRecalcDurationMs": 19.424999999999997,
      "layouts": 0,
      "layoutDurationMs": 0,
      "taskDurationMs": 384.164,
      "heapDeltaBytes": -4266324
    },
    {
      "name": "canvas-mouse-sweep",
      "durationMs": 2060.6190000000024,
      "styleRecalcs": 184,
      "styleRecalcDurationMs": 54.905,
      "layouts": 12,
      "layoutDurationMs": 3.5599999999999996,
      "taskDurationMs": 1054.656,
      "heapDeltaBytes": -3933200
    },
    {
      "name": "canvas-mouse-sweep",
      "durationMs": 2030.5979999999977,
      "styleRecalcs": 181,
      "styleRecalcDurationMs": 47.72,
      "layouts": 12,
      "layoutDurationMs": 3.2980000000000005,
      "taskDurationMs": 999.1970000000001,
      "heapDeltaBytes": -2944144
    },
    {
      "name": "canvas-mouse-sweep",
      "durationMs": 2012.0400000000132,
      "styleRecalcs": 180,
      "styleRecalcDurationMs": 46.665,
      "layouts": 12,
      "layoutDurationMs": 2.9970000000000003,
      "taskDurationMs": 992.4920000000001,
      "heapDeltaBytes": -3998332
    },
    {
      "name": "dom-widget-clipping",
      "durationMs": 625.3799999999785,
      "styleRecalcs": 45,
      "styleRecalcDurationMs": 16.793,
      "layouts": 0,
      "layoutDurationMs": 0,
      "taskDurationMs": 392.69,
      "heapDeltaBytes": 7576952
    },
    {
      "name": "dom-widget-clipping",
      "durationMs": 548.0640000000108,
      "styleRecalcs": 41,
      "styleRecalcDurationMs": 12.616,
      "layouts": 0,
      "layoutDurationMs": 0,
      "taskDurationMs": 341.132,
      "heapDeltaBytes": 7638848
    },
    {
      "name": "dom-widget-clipping",
      "durationMs": 608.5399999999481,
      "styleRecalcs": 45,
      "styleRecalcDurationMs": 12.379999999999999,
      "layouts": 0,
      "layoutDurationMs": 0,
      "taskDurationMs": 368.78999999999996,
      "heapDeltaBytes": 7402580
    },
    {
      "name": "subgraph-dom-widget-clipping",
      "durationMs": 600.6720000000314,
      "styleRecalcs": 73,
      "styleRecalcDurationMs": 15.611,
      "layouts": 0,
      "layoutDurationMs": 0,
      "taskDurationMs": 421.04400000000004,
      "heapDeltaBytes": 15436880
    },
    {
      "name": "subgraph-dom-widget-clipping",
      "durationMs": 584.1409999999883,
      "styleRecalcs": 72,
      "styleRecalcDurationMs": 13.912999999999998,
      "layouts": 0,
      "layoutDurationMs": 0,
      "taskDurationMs": 394.661,
      "heapDeltaBytes": 14678212
    },
    {
      "name": "subgraph-dom-widget-clipping",
      "durationMs": 663.0040000000008,
      "styleRecalcs": 79,
      "styleRecalcDurationMs": 20.659999999999997,
      "layouts": 1,
      "layoutDurationMs": 0.2930000000000001,
      "taskDurationMs": 444.38800000000003,
      "heapDeltaBytes": 15925020
    },
    {
      "name": "subgraph-idle",
      "durationMs": 1997.4980000000073,
      "styleRecalcs": 121,
      "styleRecalcDurationMs": 19.794000000000004,
      "layouts": 0,
      "layoutDurationMs": 0,
      "taskDurationMs": 378.9170000000001,
      "heapDeltaBytes": -3769056
    },
    {
      "name": "subgraph-idle",
      "durationMs": 1995.1650000000427,
      "styleRecalcs": 121,
      "styleRecalcDurationMs": 17.837,
      "layouts": 0,
      "layoutDurationMs": 0,
      "taskDurationMs": 355.759,
      "heapDeltaBytes": -4677012
    },
    {
      "name": "subgraph-idle",
      "durationMs": 1995.0840000000198,
      "styleRecalcs": 120,
      "styleRecalcDurationMs": 18.319000000000003,
      "layouts": 0,
      "layoutDurationMs": 0,
      "taskDurationMs": 368.787,
      "heapDeltaBytes": -3731688
    },
    {
      "name": "subgraph-mouse-sweep",
      "durationMs": 1717.6320000000374,
      "styleRecalcs": 156,
      "styleRecalcDurationMs": 49.717,
      "layouts": 16,
      "layoutDurationMs": 4.011000000000001,
      "taskDurationMs": 807.852,
      "heapDeltaBytes": -6521788
    },
    {
      "name": "subgraph-mouse-sweep",
      "durationMs": 1989.4770000000221,
      "styleRecalcs": 172,
      "styleRecalcDurationMs": 49.43300000000001,
      "layouts": 16,
      "layoutDurationMs": 4.772,
      "taskDurationMs": 942.7289999999999,
      "heapDeltaBytes": -5103624
    },
    {
      "name": "subgraph-mouse-sweep",
      "durationMs": 1731.4119999999775,
      "styleRecalcs": 157,
      "styleRecalcDurationMs": 51.522,
      "layouts": 16,
      "layoutDurationMs": 4.535,
      "taskDurationMs": 803.549,
      "heapDeltaBytes": -5311712
    }
  ]
}

@AustinMroz AustinMroz merged commit 0596e36 into core/1.41 Mar 9, 2026
39 checks passed
@AustinMroz AustinMroz deleted the backport-9671-to-core-1.41 branch March 9, 2026 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport Backporting a PR onto a release candidate size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants