-
Notifications
You must be signed in to change notification settings - Fork 0
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
refactor: favicon color-based border #203
Conversation
@@ -0,0 +1,7 @@ | |||
export const convertHexToRGBA = (hex: string, opacity: string): string => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please add a utils testcase for this as well
> | ||
<img className='h-full w-full object-contain' src={appLogo} alt={alt || appName} /> | ||
</div> | ||
<Color src={appLogo} crossOrigin='anonymous' format='hex'> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this doesn't seem to apply to the "connected apps" list as it still has the grey background there at all times
[connections] change circle color based on favicon color
Summary
color-thief-react
has been installed and implemented to get the predominant color from the favicon to implement it in the border.convertToRgba
util method to apply opacity to the hex color we get from color-thief.Checklist
pnpm dev
andpnpm dev:bare
work as intended