-
-
Notifications
You must be signed in to change notification settings - Fork 385
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
Transactions Page Follow-Up #4779
Conversation
Your Render PR Server URL is https://oc-styleguide-pr-4779.onrender.com. Follow its progress at https://dashboard.render.com/static/srv-bsn07vqpp1jte7h1ftmg. |
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/opencollective/opencollective-frontend/fgjiiy3n1 |
3c81048
to
4d3a1a0
Compare
}; | ||
|
||
StyledLink.defaultProps = { | ||
colorShade: 500, |
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.
I don't get why we need to customize this prop, links should normally always use the same color no?
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.
It was hardcoded to be 500 and 300 on hover.
I added a property to customize so I can update the colors if the background is a bit darker, like the collapsable transaction details where the background is grey.
4d3a1a0
to
5e43ee4
Compare
Related to opencollective/opencollective#3385
Requires opencollective/opencollective-api#4346