nft-preview-card-component-main

Community feedback

• 

  Rio Cantre•9,690

  @RioCantrePosted almost 3 years ago

  Hello there! Good job in completing this challenge! Looking at your solution, I would suggest the following for you...

  • Wrap the content of container with specific tag like main. For HTML structures, refer it with this one Semantics
  • Include description with the alt in img tags
  • Instead of inline style, import it on the CSS file
  • Fix the width size into width: 330px; in the .container. Adjust the margin into margin: 8rem auto;
  • Add cursor: pointer; in .aimg rule set. Adjust margin into margin: 6% 0;
  • Add color: hsl(215, 51%, 70%); and adjust margin into margin: 0 auto; in hr rule set
  • Add border: 1px solid white; and border-radius: 50%; in .three img rule set
  • Modify padding in .padding-class into padding: 0 6% 6% 6%;

  Above all, good job with this one. Hope this helps and Keep it up!

  Marked as helpful

  1

  ---

  

  Mohamed Wagdy•240

  @MohamedWagdy7Posted almost 3 years ago

  @RioCantre Thanks a lot, I got huge benefits from your review.

  0

  

  Mohamed Wagdy•240

  @MohamedWagdy7Posted almost 3 years ago

  @RioCantre I hope u can help me with this "error 404" that appears in this solution and doesn't appear when I open the GitHub view link.

  0

  

  Rio Cantre•9,690

  @RioCantrePosted almost 3 years ago

  @MohamedWagdy7 I deployed my projects with Netlify. I haven't tried Github pages yet.

  0