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

feat: Added Hue XY Conversions #155

Merged
merged 4 commits into from
Sep 22, 2024
Merged

feat: Added Hue XY Conversions #155

merged 4 commits into from
Sep 22, 2024

Conversation

Tynopia
Copy link
Contributor

@Tynopia Tynopia commented Sep 22, 2024

Hey there! 😃

First of all, I want to express my appreciation for this amazing repository, it has saved me hours of work!

I am currently working on a Philips Hue Controller that can send the XY color values derived from the HEX values provided by your UI. To achieve this, I referred to the official Philips documentation and implemented the following XY conversion functions:

  • hexToXY
  • xyToHex
  • rgbToXY
  • xyToRgb

I hope you find these additions useful!

@jaywcjlove
Copy link
Member

@Tynopia Thank you for your PR! I took some time to learn about XY color values after reviewing your submission, and I think it's a great addition. I’ll go ahead and merge it!

@Tynopia
Copy link
Contributor Author

Tynopia commented Sep 22, 2024

Thank you 🎊

I’m making some updates to the documentation and realized I forgot to include the color() function. So i will also add this

@jaywcjlove
Copy link
Member

@Tynopia
Copy link
Contributor Author

Tynopia commented Sep 22, 2024

Yeah, give me 10 min 👍

@Tynopia
Copy link
Contributor Author

Tynopia commented Sep 22, 2024

Oops, I'll never say 10 minutes again if it takes 3 hours 😆 . I now have a fix and everything should work now
@jaywcjlove

@Tynopia
Copy link
Contributor Author

Tynopia commented Sep 22, 2024

Fixed

@jaywcjlove jaywcjlove merged commit 8643165 into uiwjs:main Sep 22, 2024
1 check passed
jaywcjlove added a commit that referenced this pull request Sep 22, 2024
github-actions bot pushed a commit that referenced this pull request Sep 22, 2024
* feat: Added Hue XY Conversions

* feat: Added Docs

* fix: Changed sRGB Logic

* fix: Added XY to Circle colors test 8643165
@jaywcjlove
Copy link
Member

@Tynopia thx!

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