cal.com
cal.com copied to clipboard
[chore]: Replace color functions
What does this PR do?
This PR uses the polished package (link) in
rgbToHex()lighten()darken()getContrast()
Fixes #8560
Environment: Production
Type of change
- Chore
- This change requires a documentation update
How should this be tested?
Checklist
- I haven't checked if my PR needs changes to the documentation
- I haven't checked if my changes generate no new warnings
The latest updates on your projects. Learn more about Vercel for Git ↗︎
| Name | Status | Preview | Comments | Updated (UTC) |
|---|---|---|---|---|
| ui | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Apr 30, 2023 6:13pm |
@SohamRatnaparkhi is attempting to deploy a commit to the cal Team on Vercel.
A member of the Team first needs to authorize it.
I have made this PR with the first changes that I felt should be made.
- The function
rgbToHex()is going unused, but I haven't currently removed it (waiting for your suggestion on it). - The functions lighten and darken, clash with the current code functions and hence I have currently made alias for the published package functions.
As I was not sure with my changes, I am awaiting your suggestions on them. Finally if required I will rebase these multiple commits and keep 1 final commit having correct changes.
The reason for the test failure has been mentioned here -> link. Please do check that out for more information regarding failing tests.
Thanks for the contribution @SohamRatnaparkhi - I actually have no clue why this doesnt work. I really expected it work better than my hacky functions :P
This PR is being marked as stale due to inactivity.