-
Notifications
You must be signed in to change notification settings - Fork 50
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
Updating the member page style according to the Figma design #203
Conversation
@develad is attempting to deploy a commit to a Personal Account owned by @Maakaf on Vercel. @Maakaf first needs to authorize it. |
<p className="text-base ml-4">{name}</p> | ||
{isAdmin && ( | ||
<Crown className="w-4 h-4 fill-[#4338CA] dark:fill-[#DBC927]" /> | ||
)} |
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.
How about we export this into a separate component? seems like something that can be used in the future across the website
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.
How about we export this into a separate component? seems like something that can be used in the future across the website
Fine by me :) you can export this into a separate component
b7b7885
to
cf39689
Compare
Also, I thought it could be a cool feature to have a fallback image in case some user will lack a profile image. |
Yes! It's a great idea. can you implement this feature? if you work on my files please pull the latest changes from my 'member-page' branch |
I don`t mind working on it but lets wait and merge this pr first to avoid conflicts |
@Darkmift |
Hey everyone!
I've updated the member page style according to the Figma design and updated the images resource address, in the current published version the member image can't be loaded and there's a console error.
Also I left the font Birzia ( I think it's looking cooler ) and left the Discord link button on mobile view.
Check out the screen shots attached: