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

@Marelyn15

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

@Abdalrzakalsouki

Posted

Hello, great solution 🎉!

however, there are some small errors that can be fixed easily

  • use alt attributes with images, <img alt="something" />
  • use level-one heading: means you can not use h2 before h1
  • I saw that you used React and Vite, you can include that in the tags
  • you can include meta description for SEO
  • specifying the width and heights for images
  • you can make the performance better by minimize the code
  • this is optional, since you used a framework, you can go with enabling PWA

Hope that is helpful, all luck!

Marked as helpful

1

@Marelyn15

Posted

@Abdalrzakalsouki Thanks a lot!!

0

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