Skip to content

Preview Of Parametric Curve Mode#1017

Draft
mmadesignerunknown wants to merge 1 commit intoCyberTimon:mainfrom
mmadesignerunknown:feat-parametric-tone-curve-with-sliders
Draft

Preview Of Parametric Curve Mode#1017
mmadesignerunknown wants to merge 1 commit intoCyberTimon:mainfrom
mmadesignerunknown:feat-parametric-tone-curve-with-sliders

Conversation

@mmadesignerunknown
Copy link
Copy Markdown
Contributor

@mmadesignerunknown mmadesignerunknown commented Apr 12, 2026

Description

Added Parametric Curve modes for RGB Channels.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Performance improvement
  • Code refactoring
  • Documentation update
  • UI/UX improvement
  • Build/CI or Dependency update

Changes Made

  • Added Parametric Curve mode for Luminance, Red, Green, Blue channels.
  • For each Parametric channel:
    • Added sliders for Shadows, Darks, Lights, Highlights
    • Added regional splitters for limiting the range(s) of the sliders' affect
    • Added reset option for splitters and sliders
    • Added copy/paste option for splitters and sliders

Screenshots/Videos

I couldn't capture/record it. But if you want to test the changes, download these files and replace them with the Curves.tsx and adjustments.tsx respectively.
Curves.tsx
adjustments.tsx

Testing

  • I have tested these changes locally and confirmed that they work as expected without issues

Test Configuration:

  • OS: Windows 10
  • Hardware: Intel i5

Checklist

  • My code follows the project's code style
  • I haven't added unnecessary AI-generated code comments
    They are added for now because it's just a preview, I will remove them once this gets finalized
  • My changes generate no new warnings or errors

Additional Notes

Only the frontend has been implemented right now, I will implement the backend once the frontend gets finalized.

AI Disclaimer:

Please state the involvement of AI in this PR:

  • This PR is entirely AI-generated
  • This PR is AI-generated but guided by a human
  • This PR was handwritten with AI assistance (spell check, logic suggestions, error resolving)
  • This PR contains only blood, sweat, and coffee (AI-free)

Inspiration:

@CyberTimon
Copy link
Copy Markdown
Owner

@mmadesignerunknown Looks good, I like the UI; I'm only worried if a +600 lines code change is really needed?

@mmadesignerunknown
Copy link
Copy Markdown
Contributor Author

@mmadesignerunknown Looks good, I like the UI; I'm only worried if a +600 lines code change is really needed?

You are right to be concerned, but the thing is that the code is AI-generated, and almost 50+ lines are just AI-comments, i might also be able to clean it up a bit more but I first wanted to get the frontend math and logic finalised before moving to next steps. Hope it helps,

Regards,

@mmadesignerunknown
Copy link
Copy Markdown
Contributor Author

Well then, I will start working on the clean-up and backend tomorrow :)

@CyberTimon
Copy link
Copy Markdown
Owner

Thanks a lot! Looking forward to it 💯

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.

2 participants