NFT Preview Card Component

Solution retrospective
Overall, I'm happy with how this turned out. I'm glad I could solve the issues in my previous challenges and actually be able to understand what each code does in the code.
I was finding it difficult to make my card size fit on the page without scrolling down or zooming out. After a night of watching tutorials, and listening to the feedback from my previous challenges. Everything became so clear and understandable.
PLEASE ANY SUGGESTION IS HIGHLY APPRECIATED. I'M LOVING THIS COMMUNITY AND THE HELP I'VE RECEIVED THROUGH YOUR FEEDBACK. Thank you
Please log in to post a comment
Log in with GitHubCommunity feedback
- @correlucas
👾Hello Dicskson Owusu, congratulations for your new solution!
🎉 Nice solution and nice code! I can see that you paid a lot of attention to your code/design. If you don’t mind I’ve some tips for you:
1.Add transitions to make smoother the interaction while the element gets hovered, you can use a value like
transition: all ease-in 0.5s
.2.Add a margin of around
margin: 20px
to avoid the card touching the screen edges while it scales down.3.The correct value for this shadow is
box-shadow: 5px 5px 15px 5px rgb(0 0 0 / 3%);
4.You can create a media query to save space in the
author section
to make each information in a different row. Here’s the code for this media query.@media (max-width: 350px) { .author { color: var(--Softblue); display: flex; font-size: 15px; align-items: center; justify-content: left; flex-direction: column; } }
✌️ I hope this helps you and happy coding!
Marked as helpful - @romila2003
Hi @ondickson,
Congratulations 🎉 for completing this challenge, your card looks great, and you have used the right semantics. Also you have no accessibility and HTML issues which is great too.
- The only thing I would suggest is to you use
min-height
property to center the card in the middle e.g.min-height: 100vh;
- Also, in the design, the word 'Equilibrium' and the profile's name has a hover effect which are shown in the design provided. I would suggest you wrap it within the
a
tag since if this was a functional NFT card, those texts would redirect you to external/internal websites.
Overall, great attempt and wish you the best for your future projects so keep coding 👍
Marked as helpful - The only thing I would suggest is to you use
- @ondickson
Thank you very much for the feedback. I will fix these issues right away.
Edit: I noticed all the minor details. got really excited and totally missed them all but I've fixed everything. I want to hear your feedback once again. That was very helpful.
Join our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord