NFT Card Component using CSS Grid and Flexbox

Solution retrospective
- Do you think the structure and design are good?
- What would you improve about my code?
I appreciate your comments in advance, greetings :)
Please log in to post a comment
Log in with GitHubCommunity feedback
- @samuel-adu
Everything seems to work as it should, good job Giuliana.
Marked as helpful - @vanzasetia
Hello there, Giuliana Saborío! 👋
Good effort on this challenge! 👍
Some recommendations from me.
- Currently, the site is missing the interactive elements. All elements that have interactivity need to be wrapped by interactive elements such as anchor tag or button tag (depending on what will happen after the users click the element).
- After that, create a custom
:focus-visible
styling for any interactive elements. This will let people who use a keyboard to navigate the site know where they are currently on the page. figure
element doesn't need to wrap the avatar and the Equilibrium image. The only reason to usefigure
is if you need to include afigcaption
.
I hope this helps! Keep up the good work! 👍
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