-
-
Notifications
You must be signed in to change notification settings - Fork 37.2k
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(client): migrate lower jaw buttons to Button component #54906
refactor(client): migrate lower jaw buttons to Button component #54906
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.
Thanks for working on this.
It may be worth to migrate all the buttons in
lower-jaw.tsx
in this PR. If so, please let me know.
Yes, I think we should migrate all buttons in the lower jaw to the Button
component 🙂
@huyenltnguyen addressed comments. However, because of these changes, I found an issue that I can't get around. Asking for feedback. |
f593fde
to
8d94a1a
Compare
Checklist:
main
branch of freeCodeCamp.Ref #54892
As suggested by @huyenltnguyen, the hover effect issue got fixed by migrating it to the new fCC
Button
component.It may be worth to migrate all the buttons in
lower-jaw.tsx
in this PR. If so, please let me know.Video demo
CleanShot.2024-05-21.at.08.19.33.mp4