product-preview-card-component-main

Community feedback

• 

  Kelvin•770

  @githukelvinPosted over 2 years ago

  Hey buddy

  Seen you havent updated the screenshot of preview

  • The margin: 5rem auto 0 auto; you can replace with margin:10rem auto; it will be exact of that of design.and that it is if you have added max-width:1440px; on the body.

  -Another feedback is the paragraph should have a font-weight:500; font-family:montserrat,sans serif. add line-height:1.5; add width:80%;

  -Another point is <main class="container"> you could have used div instead of the landmark. add alt="" on your cart image accesibility respect heading from <h1></h1> to <h6></h6 in that order.

  Thats all @afaiz-space Happy coding Hope it is Helpful

  2
• 

  Adebisi•350

  @adebisiabdussamadPosted over 2 years ago

  Cheers 🥂 on completing this challenge.. .

  An img element must have an alt attribute. This enables physical challenge to understand the image better.

  This is your code is:<img class=cart_logo src=images/icon-cart.svg><span> This is how it should have been:<img class=cart_logo src=images/icon-cart.svg alt="the picture in text"><span>

  your code is really amazing...hope you find this helpful... Happy coding!!!👌👍

  1