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-component

Gustavo 70

@gunney69

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

@pinhob

Posted

Hi, Gustavo! Great work finishing the challenge! 🚀

I have a few suggestions for you of things that I found helpful doing my solution for this challenge:

  • I like your approach to the hover effect in the NFT image, but try to use width: 100% in the .imgNFT class. This will make your image responsible and also the same size as the ::before and ::after elements.

  • You can remove the height of the <main> component, or use min-height: 60vh instead of height: 60vh. This way, the element will grow to fit all the children's content. You can see more about the min-height property here, it's really helpful in some cases.

  • Take a look at the BEM methodology to name classes. It's a great way to choose how to name our classes, in my opinion! (If you want a really good video in Portuguese, watch this one)

I hope you find theses tips helpful.

Congrats again for the challenge and happy coding! 🤓

1

Gustavo 70

@gunney69

Posted

Thanks for support @pinhob

1

@pinhob

Posted

Anytime, @gunney69!

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