Skip to content
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

Contributors' data through GitHub API and Contributors' Card Design #13

Merged
merged 5 commits into from
Oct 28, 2022

Conversation

BadgerBloke
Copy link
Contributor

@BadgerBloke BadgerBloke commented Oct 27, 2022

Getting contributors' data through GitHub API #7
Updated the contributors' card design #11

@netlify
Copy link

netlify bot commented Oct 27, 2022

Deploy Preview for designerslobby ready!

Name Link
🔨 Latest commit e148003
🔍 Latest deploy log https://app.netlify.com/sites/designerslobby/deploys/635b5a7fa025bb0008b617a6
😎 Deploy Preview https://deploy-preview-13--designerslobby.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@singhnitin77
Copy link
Owner

Would be great if we can reduce the height and width of the card, work on flex and grid.

https://gradientify.com/contributors - similar to this flex and grid accommodating 4 cards in one row + responsive. You can also implement the similar design too for the cards.

@BadgerBloke
Copy link
Contributor Author

Would be great if we can reduce the height and width of the card, work on flex and grid.

https://gradientify.com/contributors - similar to this flex and grid accommodating 4 cards in one row + responsive. You can also implement the similar design too for the cards.

sure

@BadgerBloke BadgerBloke changed the title Contributors data through GitHub API Contributors' data through GitHub API and Contributors' Card Design Oct 28, 2022
@BadgerBloke
Copy link
Contributor Author

I have updated the contributors' card design. @singhnitin77 Please have a look at the same.
1

@singhnitin77
Copy link
Owner

@CA-MKSingh Good job 👏👏, everything looks great.

@singhnitin77 singhnitin77 merged commit 9af202c into singhnitin77:main Oct 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants