nft preview card component main

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

• Use the same closing tag for every tag, like this one <h3>Equilibrium #3429</h2> . Remove either h3 or h2 and use the same for the opening and closing tags.
• It's common to use div to wrap the whole content, instead use a specific one like main to wrap the container for readability
• Include the font-family either inside the body or container tag
• Add cursor:pointer for the hover state of the design
• Adjust the padding of the card into 1.5em

Hope this helps and Keep up the good work!

Hi, @MatiX221 ~

Congratulate on your solution to the challenge on FM platform. I have studied your work carefully and learned a lot from it.

Here are some of the tips I like to provide.

• On smaller devices, card goes over screen, and I think max-width: 100% can solve this issue.
• I suggest you to try transition on hoverable elements like image, heading and creator name.
• Please try BEM for naming element classes. It will help you a lot on bigger projects.

https://www.frontendmentor.io/solutions/my-first-solution-on-chanllenge-V-4IzAivH

Here is my solution to this challenge, and if it can help you even a bit, it would be happy to me.

Cheers ~