Skip to content

Add card component #7

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

Merged
merged 4 commits into from
Sep 27, 2023
Merged

Add card component #7

merged 4 commits into from
Sep 27, 2023

Conversation

captainmangoC4
Copy link
Collaborator

@captainmangoC4 captainmangoC4 commented Sep 26, 2023

This generic card component can be used for the contest tile, the reports tile, and for the team members tab in the new team profile

@vercel
Copy link

vercel bot commented Sep 26, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
components-library ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 27, 2023 5:23pm

Copy link
Contributor

@leo95oliveira leo95oliveira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This component looks awesome @captainmangoC4! I've requested some very minor changes but other than that it should be ready to go! Once the changes have been made just let me know and I'll approve the PR 😄

On a side note, looking at the component on Storybook, I noticed that it seems to have a fixed width; did you add any CSS related to responsiveness for the component? Just asking because if you did add it and it's not reproducing properly on Storybook, I might have to look into it to see what the issue is.

Copy link
Contributor

@leo95oliveira leo95oliveira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🚀

@captainmangoC4 captainmangoC4 merged commit 31d2153 into develop Sep 27, 2023
@captainmangoC4 captainmangoC4 deleted the mango/card branch September 27, 2023 18:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants