Skip to content

[backport core/1.34] fix: Note/MarkdownNote node color change not reactive in vueNodes mode#7502

Merged
christian-byrne merged 1 commit intocore/1.34from
backport-7294-to-core-1.34
Dec 15, 2025
Merged

[backport core/1.34] fix: Note/MarkdownNote node color change not reactive in vueNodes mode#7502
christian-byrne merged 1 commit intocore/1.34from
backport-7294-to-core-1.34

Conversation

@comfy-pr-bot
Copy link
Member

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

Backport of #7294 to core/1.34

Automatically created by backport workflow.

┆Issue is synchronized with this Notion page by Unito

#7294)

## Summary

Move color/bgcolor initialization from class field overrides to
constructor to preserve LGraphNodeProperties getter/setter
instrumentation.

Class field overrides were replacing the reactive property descriptors
set by the parent constructor, preventing change events from firing.

issue found while tesing in
#3449

## Screenshots
Before


https://github.com/user-attachments/assets/04499a3a-15c2-44fd-9819-6dd5f6849f20


After


https://github.com/user-attachments/assets/ba93278b-9761-4d45-abb3-2a57ff95a900

┆Issue is synchronized with this [Notion
page](https://www.notion.so/PR-7294-fix-Note-MarkdownNote-node-color-change-not-reactive-in-vueNodes-mode-2c46d73d3650818f8ee6f6f0c0e61d39)
by [Unito](https://www.unito.io)
@comfy-pr-bot comfy-pr-bot requested a review from a team as a code owner December 15, 2025 18:21
@comfy-pr-bot comfy-pr-bot added the backport Backporting a PR onto a release candidate label Dec 15, 2025
@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Dec 15, 2025
@github-actions
Copy link

github-actions bot commented Dec 15, 2025

🎭 Playwright Test Results

Some tests failed

⏰ Completed at: 12/15/2025, 07:10:53 PM UTC

📈 Summary

  • Total Tests: 484
  • Passed: 465 ✅
  • Failed: 1 ❌
  • Flaky: 8 ⚠️
  • Skipped: 10 ⏭️

📊 Test Reports by Browser

  • chromium: View Report • ✅ 462 / ❌ 1 / ⚠️ 8 / ⏭️ 10
  • chromium-2x: View Report • ✅ 2 / ❌ 0 / ⚠️ 0 / ⏭️ 0
  • chromium-0.5x: View Report • ✅ 1 / ❌ 0 / ⚠️ 0 / ⏭️ 0
  • mobile-chrome: Deployment failed

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

@github-actions
Copy link

github-actions bot commented Dec 15, 2025

🎨 Storybook Build Status

Build completed successfully!

⏰ Completed at: 12/15/2025, 06:23:28 PM UTC

🔗 Links


🎉 Your Storybook is ready for review!

@christian-byrne christian-byrne merged commit d1a95e2 into core/1.34 Dec 15, 2025
31 of 33 checks passed
@christian-byrne christian-byrne deleted the backport-7294-to-core-1.34 branch December 15, 2025 19:30
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:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants