Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

product-preview-card-component-main

#bootstrap
faizanโ€ข 2,420

@afaiz-space

Desktop design screenshot for the Product preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


plz, gives feedback

Community feedback

Kelvinโ€ข 770

@githukelvin

Posted

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โ€ข 330

@adebisiabdussamad

Posted

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

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord