-
-
Notifications
You must be signed in to change notification settings - Fork 120
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
Bill Cosponsor Card #1244
base: main
Are you sure you want to change the base?
Bill Cosponsor Card #1244
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@akh4rf is attempting to deploy a commit to the Maple Testimony Team on Vercel. A member of the Team first needs to authorize it. |
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.
Hey Austin! Nice work so far!
Just a couple broad notes:
-
Look at building the graph elements as react components. And/or checkout react-chartjs library as an option.
-
The role-over popup is a great. Let's wait to do any more on it until we get input from the design team.
-
Work on the styling and the other sponsor card elements and states to bring it inline with the figma (figma section)
Summary
#832
VIdeo
Screen.Recording.2023-08-07.at.2.22.18.AM.mov
Checklist
Screenshots
Add some screenshots highlighting your changes.
Known issues
If you've run against limitations or caveats, include them here. Include follow-up issues as well.
Steps to test/reproduce
yarn storybook