Skip to content

[backport core/1.35] fix: prevent custom context menu when editing text#7636

Merged
christian-byrne merged 1 commit intocore/1.35from
backport-7633-to-core-1.35
Dec 20, 2025
Merged

[backport core/1.35] fix: prevent custom context menu when editing text#7636
christian-byrne merged 1 commit intocore/1.35from
backport-7633-to-core-1.35

Conversation

@comfy-pr-bot
Copy link
Member

@comfy-pr-bot comfy-pr-bot commented Dec 19, 2025

Backport of #7633 to core/1.35

Automatically created by backport workflow.

┆Issue is synchronized with this Notion page by Unito

## Summary
- Stop contextmenu event propagation in EditableText component
- Allows browser's native context menu (copy/paste) when renaming nodes

## Test plan
- [ ] Double-click a node title to enter edit mode
- [ ] Select text and right-click
- [ ] Verify browser's native context menu appears (not the node context
menu)

┆Issue is synchronized with this [Notion
page](https://www.notion.so/PR-7633-fix-prevent-custom-context-menu-when-editing-text-2ce6d73d365081e38461d080abe12b32)
by [Unito](https://www.unito.io)
@comfy-pr-bot comfy-pr-bot added the backport Backporting a PR onto a release candidate label Dec 19, 2025
@comfy-pr-bot comfy-pr-bot requested a review from a team as a code owner December 19, 2025 04:48
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Dec 19, 2025
@github-actions
Copy link

github-actions bot commented Dec 19, 2025

🎨 Storybook Build Status

Build completed successfully!

⏰ Completed at: 12/19/2025, 04:50:22 AM UTC

🔗 Links


🎉 Your Storybook is ready for review!

@github-actions
Copy link

github-actions bot commented Dec 19, 2025

🎭 Playwright Test Results

🕵🏻 No test results found

⏰ Completed at: 12/19/2025, 10:54:14 AM UTC

📊 Test Reports by Browser

  • chromium: Deployment failed
  • chromium-2x: Deployment failed
  • chromium-0.5x: Deployment failed
  • mobile-chrome: Deployment failed

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

@christian-byrne christian-byrne merged commit 8c651f2 into core/1.35 Dec 20, 2025
21 of 33 checks passed
@christian-byrne christian-byrne deleted the backport-7633-to-core-1.35 branch December 20, 2025 00:21
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:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments