Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Gradient API (v2) #1063

Draft
wants to merge 1 commit into
base: main/4
Choose a base branch
from

Conversation

thiccaxe
Copy link

@thiccaxe thiccaxe commented Apr 5, 2024

Addresses some of the main concerns of my previous rendition, mainly separating the concept of a gradient and the interpolation of colors.

By not requiring colors to be a TextColor the API interface is greatly simplified (far less conversions from rgb and back). However, this requires an additional step on the user's end to convert from whatever color space they are using (say, HSV, or perhaps, (ok)LAB)

I feel that this interface is more versatile and easier for users to extend on their end.

  • gradient API
  • update minimessage to use this API
    • gradients
      • stops?
    • rainbows
  • color spaces
    • RGB
    • HSV (need to verify implementation)
    • OkLab
    • OkLCH
  • tests
  • gradient with stops
  • RGB
  • HSV
  • OKLAB
  • OKLCH
  • minimessage

POC - some code untested
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants