nft-preview

Community feedback

• 

  Chanda•800

  @Chanda-AbdulPosted 5 months ago

  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