Skip to content
Submitted 7 months ago

Product-preview-card-component

kirti253•20
@kirti253
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Product preview card component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Community feedback

  • Mukarram Haq•270
    @MukarramHaqPosted 6 months ago

    Hey, great work!

    I would like to point out a few things:

    HTML:

    1. In your <body> put everything inside the <main> tag for this particular project. It specifies the main content of the page.
    2. Always have something in you alt for an image. You can say something like Chanel Parfum
    3. <br> tags should be avoided write the whole sentence in one line.
    4. In your <button> tag you can give the source of the SVG that you downloaded with this project it should be images/icon-cart.svg

    CSS:

    1. In .image never use px for width. Always your rem.
    2. The font-size should never be in px use rem for that as well
    3. In button remove the borders by border:none.

    This is all I could help you with with my limited knowledge. I hope you find this useful :)

    0

Please log in to post a comment

Log in with GitHub

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