Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

NFT preview card

@dsaglam94

Desktop design screenshot for the NFT preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

Rio Cantre 9,690

@RioCantre

Posted

Hello there! Nice work with this challenge. Looking at your solution, I would suggest the following for you...

  • Instead of using section, wrap the whole content card with specific tag like main
  • Include description with the alt in img tag
  • Clean the whitespaces in the code
  • Remove unused code to keep it clean
  • Adjust the width of the .divider into 330px
  • Add border: 1px solid white; and border-radius: 50%; in the .avatar-img rule set

Overall, you did great with the design. Hope this helps and Keep it up!

Marked as helpful

1

@dsaglam94

Posted

@RioCantre Hey there! Thank you very much for your informative reply. I really appreciate it! I checked the parts you mentioned and fixed them. It turned out better.

1
Rio Cantre 9,690

@RioCantre

Posted

@dsaglam94 Awesome! I'm glad it was helpful. Cheers!

1

Please log in to post a comment

Log in with GitHub
Discord logo

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