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

Responsive NFT preview card component

Joshua 130

@dyntbn

Desktop design screenshot for the NFT preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


As a bonus to this challenge, I made the image popup on click of the main image.

Community feedback

Travolgi 🍕 31,420

@denielden

Posted

Hi Joshua, very great work! 😉

Using <hr> for the line is not the best way because this tag have a semantic meaning... in this case use div with border-bottom because this line is decorative.

Also instead of using px use relative units of measurement like rem -> read here

Hope this help! 😁

Marked as helpful

1

Joshua 130

@dyntbn

Posted

@denielden

Reading about the <hr> on MDN docs, I see now that it did not fit semantically. I will fix that and see what I can do to minimize my use of px units.

Thanks for pointing these out Deniel! 🙂

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