-
Notifications
You must be signed in to change notification settings - Fork 30
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(Button): Update hover background #2934
Conversation
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.
lgtm! while reviewing the button figma i noticed the StrokeButton bg hover color is also currently off (it should be the same color as TextButton's is here). could ya include that change in this work or i can cut a follow-up ticket for it
@dreamwasp i can do it here! |
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.
nice! i was double-checking the figma and the color still looks a little off to me so i compared side-by-side against a beige background.
i narrowed it down to this &:after
effect here which is shading everything a little darker -
[ButtonSelectors.SHADOW]: { |
i didn't look into it that much, but we can prob use these just in the button styles they're applicable to and move the animation transitions to the background colors for the stroke + textbutton
📬Published Alpha Packages:@codecademy/[email protected] |
🚀 Styleguide deploy preview ready! |
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.
perfection
Overview
Updates all variants of
Button
hover background to be the correct values with a nice transitionPR Checklist
Testing Instructions
PR Links and Envs