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

svo15 230

@svo15

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

Chanda 800

@Chanda-Abdul

Posted

Good job on your NFT card component solution!

A few sugesstions

  • consider using less <div>'s for html semantic purposes. <div>'s should only be used for styling purposes and not as containers.

    • This article expains this concept well An intro to semantic HTML

    • ex. instead of <div class="container"> just use <main>

    • instead of <div class="price"><h2> use <h2 class="price">

  • the active states are not working, you could add something like this

a:hover {
      color: white;
      cursor: pointer;
    }
  • define font-family in the body selector
body {
    font-family: 'Outfit', sans-serif;
}
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