community/members: fix overflow in ProfileCard #6542
Conversation
|
🚀 Preview for commit 96a0bc4 at: https://684ade2053b72c8135fe0776--layer5.netlify.app |
All ProfileCards are now the same width and height. This is not possible with flexbox, so it has been changed to use grid. The Makefile has also been updated to use `npx gatsby ...` instead of `gatsby ...`. This removes the need for installing gatsby-cli globally. Additionally, there was a stray meshmate_of_the_year svg which was not being used anywhere, and has been removed. Signed-off-by: Pranjal Kole <pranjal.kole7@gmail.com>
pranjalkole
force-pushed
the
fix-community/members
branch
from
96a0bc4 to
ef947de
Compare
|
🚀 Preview for commit ef947de at: https://684ae2eeb82722792fdf17c0--layer5.netlify.app |
|
@pranjalkole Thank you for your contribution! Add it as an agenda item to the meeting minutes, if you would :) |
|
🚀 Preview for commit ceb95c7 at: https://684ffed6bc67c140431fe0d6--layer5.netlify.app |
|
Suggestion : Can we go ahead with showing full name when hover and increase the size of box with it too. and For normal view we can use First Name. |
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
stale
bot
added
the
issue/stale
|
@pranjalkole any updates? |
stale
bot
removed
the
issue/stale
|
@vr-varad will you take over this PR? @pranjalkole has abandoned the effort here. Either finish or close it and finish separately, please. |
|
This is true for any issue assigned to @pranjalkole or PR authored by him. |
|
I guess someone has resolved the issue, so closing this one. |
All ProfileCards are now the same width and height. This is not possible with flexbox, so it has been changed to use grid.
The Makefile has also been updated to use
npx gatsby ...instead ofgatsby .... This removes the need for installing gatsby-cli globally. Additionally, there was a stray meshmate_of_the_year svg which was not being used anywhere, and has been removed.Description
This PR fixes the following issue:
After the changes:
Notes for Reviewers
Signed commits