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

Add types for color-rgba and color-space/* #16001

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

MoonE
Copy link
Contributor

@MoonE MoonE commented Jul 18, 2024

Types for color-rgba are available on npm, color-space is not, so I add custom type declarations for the functions used.

Copy link

📦 Preview the website for this branch here: https://deploy-preview-16001--ol-site.netlify.app/.

@ahocevar
Copy link
Member

@MoonE Thanks for your effort! I think more users could benefit from the color-space types if you submitted them to Definitely Typed instead of adding them here.

@MoonE
Copy link
Contributor Author

MoonE commented Jul 18, 2024

Yeah, I thought about that.
There are huge amount of functions, do you feel like they'd accept type declaration for just a few functions? I didn't want to create types for all.

@ahocevar
Copy link
Member

@MoonE, yes I think they'll accept a pull request with only a subset of the signatures typed. Better than nothing!

@MoonE
Copy link
Contributor Author

MoonE commented Jul 25, 2024

I made a pull request in colorjs/color-space#57

@ahocevar
Copy link
Member

Thanks, @MoonE !

@ahocevar
Copy link
Member

ahocevar commented Jan 9, 2025

I made a new attempt to get types for color-parse: colorjs/color-space#58.

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