nft preview card component

Community feedback

• 

  Fury•945

  @ashiqfuryPosted almost 3 years ago

  Good Job 😘

  • Use some transition in parent elements, it looks the hovering animations pleasent.
  • Images must have some width and height properties. It prevents the shrink effect when responsiveness.
  • Use font api links in HTML page instead of importing in css page, it improves your site performance.

  Build some projects like this in future. I am eagerly waiting for your projects. I hope my suggestions will help you.

  Thank you so much.

  Pakoda vera level pandra pooo... 🔥😍 (Only this user can understand this 😁)

  Marked as helpful

  1

  ---

  

  Ray•40

  @RizwanaRayPosted almost 3 years ago

  @ashiqfury To: ennouda pakoda❤... Thankyou for your kind support sir..i'll try your tips ...

  143<3

  0
• 

  Rio Cantre•9,690

  @RioCantrePosted almost 3 years ago

  Hello there! Good job in completing this project. Viewing at your solution, I think you should considered the following as well...

  • Wrap the whole content with specific tag like main . The attribution can be wrap with footer tag. For HTML structures, refer it with this one Semantics
  • Include alt with description in image tags , like this line <img src="./images/icon-view.svg"/>
  • Pay attention in using / in the directory of the images, like this one <div class="container"><img src=".\images\image-equilibrium.jpg" alt="Equilibrium.jpg" >, turn it into <div class="container"><img src="./images/image-equilibrium.jpg" alt="Equilibrium.jpg" >
  • Add margin-top: 3.5rem; in .box rule set
  • Add padding: 1rem; in the attribution rule set
  • Set the .box .avator color into color: var(--Softblue); and .white span into color: var(--White); and padding: 0.3rem;
  • Adjust the height of the hero image hover state in the media query

  Above all, you did great! Hope this helps and Keep it going!

  Marked as helpful

  0