Nft preview card

Community feedback

• 

  Lucas 👾•104,340

  @correlucasPosted over 2 years ago

  👾 Hello Makha, congratulations for your solution!

  The overall design is all fine, but there's some stuff a bit different from the reference design. I'll give you some tips:

  1.The card component is really small in comparison to the original image. You've used width: 240px; try to use a value around max-width: 350px;.

  2.The paragraph inside the card isn't gray, is white/blue color, you can see these color values inside the folder started files > styleguide.md. The correct color is: hsl(215, 51%, 70%);.

  3.You don't need the "contain" div, instead of that, apply the same class to the <main> to clean your code.

  3.Add a single padding for the whole container to create the space between the inner elements, you don't need to set padding for each element (img, h1, icons).

  Hope it helps you bro, keep it up!

  Marked as helpful

  1
• 

  Divine Obeten•2,415

  @Deevyn9Posted over 2 years ago

  Hi Makha, congrats on completing this challenge, you’ll need to make the container responsive,

  Happy coding 🎈

  2
• 

  M Irfan Ashraf•60

  @IrfanAshraf-proPosted over 2 years ago

  Hi, the card looks really great and in addition you have used some filters on the images when hover. That was good. :)

  1