NFT preview card component

Community feedback

• 

  Macaulay Omoikhoje De-great•550

  @MacChristoPosted over 2 years ago

  Give the card title Equilibrium #3429 a different class name. like: <h1 class="card-title">Equilibrium #3429</h1> <----add----

  1. The card title Equilibrium #3429 should have a hover element with the color set to cyan. .card-title:hover { color: hsl(178, 100%, 50%); } <----add-----

  2. Also in this, <a href="https://twitter.com/@oyeinbomo">Jules Wyvern</a> <----Remove---- remove it from the anchor tag and place it in a span tag and give it a class name like: <span class="avatar-name">  Jules Wyvern </span>. <----add----

  3. Then on hover, set the color to cyan: .avatar-name:hover { color: hsl(178, 100%, 50%); }<----add----

  Hope this help!... Keep coding👍

  Marked as helpful

  0

  ---

  

  Oyeinbomo•160

  @Oyeins-GUIPosted over 2 years ago

  @MacChristo Thanks man! I actually forgot about all these.

  0

  

  Macaulay Omoikhoje De-great•550

  @MacChristoPosted over 2 years ago

  @Oyeins-GUI happy it was helpful 👍

  0
• 

  Ahmed Bayoumi•6,740

  @Bayoumi-devPosted over 2 years ago

  Hey! It looks good!... Here are some suggestions:

  • Document should have one main landmark, Contain the component with <main>.

  <main>
     <div class="nft-card">
        //...
     </div>
  </main>
  

  • All page content should be contained by landmarks, Contain the attribution with <footer>.

  <footer>
     <div class="attribution">
        //...
     </div>
  </footer>
  

  • When hovering over the card title Equilibrium #3429 and the creator name Creation of Jules Wyvern the color should be changed to cyan color as recommended in this challenge.

  .card-details h1:hover {
     color: var(--cyan);
  }
  

  Hope this help!... Keep coding👍

  Marked as helpful

  0

  ---

  

  Oyeinbomo•160

  @Oyeins-GUIPosted over 2 years ago

  @Bayoumi-dev Thanks man! This was helpful.

  0