Skip to content

Conversation

@comfy-pr-bot
Copy link
Member

@comfy-pr-bot comfy-pr-bot commented Sep 29, 2025

Backport of #5846 to core/1.27

Automatically created by backport workflow.

┆Issue is synchronized with this Notion page by Unito

## Summary

Fixes the laggy textarea select noted in Discord.

## Changes

- **What**: Fixes the case where a DomWidget that was repeatedly made
visible accumulated as many event listeners as times it was toggled on.

┆Issue is synchronized with this [Notion
page](https://www.notion.so/PR-5846-fix-Only-add-the-listeners-for-DomWidget-components-once-27d6d73d3650818382e0d0c565fc862a)
by [Unito](https://www.unito.io)
@comfy-pr-bot comfy-pr-bot added the backport Backporting a PR onto a release candidate label Sep 29, 2025
@comfy-pr-bot comfy-pr-bot requested a review from a team as a code owner September 29, 2025 18:54
@github-actions
Copy link

github-actions bot commented Sep 29, 2025

🎭 Playwright Test Results

Some tests failed

⏰ Completed at: 09/29/2025, 07:05:55 PM UTC

📈 Summary

  • Total Tests: 448
  • Passed: 417 ✅
  • Failed: 1 ❌
  • Flaky: 1 ⚠️
  • Skipped: 29 ⏭️

📊 Test Reports by Browser

  • chromium: View Report • ✅ 410 / ❌ 1 / ⚠️ 1 / ⏭️ 29
  • chromium-2x: View Report • ✅ 2 / ❌ 0 / ⚠️ 0 / ⏭️ 0
  • chromium-0.5x: View Report • ✅ 1 / ❌ 0 / ⚠️ 0 / ⏭️ 0
  • mobile-chrome: View Report • ✅ 4 / ❌ 0 / ⚠️ 0 / ⏭️ 0

🎉 Click on the links above to view detailed test results for each browser configuration.

AustinMroz
AustinMroz previously approved these changes Sep 29, 2025
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Sep 29, 2025
@AustinMroz
Copy link
Collaborator

Test failure expected. See #5839

@AustinMroz AustinMroz enabled auto-merge (squash) September 29, 2025 19:25
@DrJKL DrJKL added the New Browser Test Expectations New browser test screenshot should be set by github action label Sep 30, 2025
@github-actions github-actions bot requested a review from a team as a code owner September 30, 2025 03:50
@christian-byrne christian-byrne merged commit cdff3f9 into core/1.27 Sep 30, 2025
2 checks passed
@christian-byrne christian-byrne deleted the backport-5846-to-1.27 branch September 30, 2025 04:20
@AustinMroz AustinMroz mentioned this pull request Sep 30, 2025
AustinMroz added a commit that referenced this pull request Sep 30, 2025
## What's Changed
### 🚀 Features
- add pricing for new api nodes (#5725)
- Rework desktop install / startup UX (#5292)

### 🐛 Bug Fixes
- Make experiment asset api setting hidden (#5861)
- Only add the listeners for DomWidget components once (#5849)
- fix invalid JSON (contains merge conflict markers) in 1.27 RC branch's
locale json (#5839)
- fix: Status indicator and close button appearing together  (#5741)
- Fix cyclic prototype errors with subgraphNodes (#5650)
- fix: don't immediately close missing nodes dialog if manager is
disabled (#5649)
- Fix SaveAs (#5644)

### 🔧 Maintenance  
- Add JSON validation CI workflow (#5838)
- Cherry-pick manager flag refactor to core/1.27 (#5646)
- Cherry-pick desktop dialogs framework to core/1.27 (#5634)

**Full Changelog**:
v1.27.5...v1.27.7

┆Issue is synchronized with this [Notion
page](https://www.notion.so/PR-5862-1-27-7-27e6d73d3650819aa712e682f6b88077)
by [Unito](https://www.unito.io)
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 New Browser Test Expectations New browser test screenshot should be set by github action size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants