Skip to content

[backport core/1.40] fix: textarea stays disabled after link disconnect on promoted widgets (#9199)#9577

Merged
christian-byrne merged 1 commit intocore/1.40from
backport-9199-to-core-1.40
Mar 8, 2026
Merged

[backport core/1.40] fix: textarea stays disabled after link disconnect on promoted widgets (#9199)#9577
christian-byrne merged 1 commit intocore/1.40from
backport-9199-to-core-1.40

Conversation

@christian-byrne
Copy link
Contributor

@christian-byrne christian-byrne commented Mar 8, 2026

Backport of #9199 to core/1.40.

Conflicts resolved in useGraphNodeManager.ts/test.ts — accepted incoming promoted widget handling changes.

Original PR: #9199
Pipeline ticket: 15e1f241-efaa-4fe5-88ca-4ccc7bfb3345

┆Issue is synchronized with this Notion page by Unito

#9199)

Fix textarea widgets staying disabled after disconnecting a link on
promoted widgets in subgraphs.

- **What**: `refreshNodeSlots` used `SafeWidgetData.name` for slot
metadata lookups, but for promoted widgets this is `sourceWidgetName`
(the interior widget name), which doesn't match the subgraph node's
input slot widget name. Added `slotName` field to `SafeWidgetData` to
track the original LiteGraph widget name, and updated `refreshNodeSlots`
to use `slotName ?? name` for correct matching.

The key change is the `slotName` field on `SafeWidgetData` — it's only
populated when `name !== widget.name` (i.e., for promoted widgets). The
`refreshNodeSlots` function now uses `widget.slotName ?? widget.name` to
look up slot metadata, ensuring promoted widgets correctly update their
`linked` state on disconnect.

Fixes #8818

┆Issue is synchronized with this [Notion
page](https://www.notion.so/PR-9199-fix-textarea-stays-disabled-after-link-disconnect-on-promoted-widgets-3126d73d3650813db499c227e6587aca)
by [Unito](https://www.unito.io)
@christian-byrne christian-byrne requested a review from a team as a code owner March 8, 2026 02:28
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Mar 8, 2026
@github-actions
Copy link

github-actions bot commented Mar 8, 2026

🎨 Storybook Build Status

Build completed successfully!

⏰ Completed at: 03/08/2026, 02:29:43 AM UTC

🔗 Links


🎉 Your Storybook is ready for review!

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 8, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: abf46c65-fe49-4aaa-8962-3c8a318b2bb9

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch backport-9199-to-core-1.40

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link

github-actions bot commented Mar 8, 2026

Playwright: 🕵🏻 0 passed, 0 failed

📊 Browser Reports
  • chromium: ❌ Deployment failed
  • chromium-2x: ❌ Deployment failed
  • chromium-0.5x: ❌ Deployment failed
  • mobile-chrome: ❌ Deployment failed

@christian-byrne christian-byrne merged commit 602784a into core/1.40 Mar 8, 2026
16 of 21 checks passed
@christian-byrne christian-byrne deleted the backport-9199-to-core-1.40 branch March 8, 2026 02:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant