Skip to content

feat: add proxyWidgetSelector to subgraph blueprints#13275

Open
jtydhr88 wants to merge 1 commit intomasterfrom
proxy-widget-selector
Open

feat: add proxyWidgetSelector to subgraph blueprints#13275
jtydhr88 wants to merge 1 commit intomasterfrom
proxy-widget-selector

Conversation

@jtydhr88
Copy link
Copy Markdown
Contributor

@jtydhr88 jtydhr88 commented Apr 3, 2026

Add channel/tone selector dropdowns to Color Curves and Color Balance blueprints so only one group of widgets is shown at a time, matching industry-standard UX (Photoshop, GIMP).
Need FE changes Comfy-Org/ComfyUI_frontend#10840

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 3, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 84f17d5b-4661-44c6-abfe-84fea2f1b947

📥 Commits

Reviewing files that changed from the base of the PR and between f21f6b2 and 7ed31f7.

📒 Files selected for processing (2)
  • blueprints/Color Balance.json
  • blueprints/Color Curves.json

📝 Walkthrough

Walkthrough

Two blueprint JSON files were updated to add proxyWidgetSelector configurations. The Color Balance blueprint now includes a "Tone" selector with three options (Shadows, Midtones, Highlights), each mapping to distinct widget value groups. The Color Curves blueprint adds a "Channel" selector with four options (RGB Master, Red, Green, Blue), each mapping to different curve editor nodes. Both selectors provide a default selection and define widget-to-control mappings. The changes enable dynamic switching between different groups of controls within each blueprint. All other graph structures, shader code, and existing configurations remain intact.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: adding proxyWidgetSelector to subgraph blueprints, which matches the actual modifications to the Color Balance and Color Curves blueprint files.
Description check ✅ Passed The description is directly related to the changeset, explaining the purpose of adding selector dropdowns to the Color Curves and Color Balance blueprints to control widget visibility, matching the actual changes made.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant