Skip to content
Submitted about 1 month ago

Product Preview Card Component

arnoldpam11•120
@arnoldpam11
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

Solution retrospective

What are you most proud of, and what would you do differently next time?

What I am most proud of is that, it didn't take too long to make this project and somehow it is responsive and kind of close to the original. I should do differently next time is to make it more stunning.

What challenges did you encounter, and how did you overcome them?

Challenges that I encountered during making this project was that I had having a hard time to set margins and paddings on the fonts because it was overflowing but then I figured it out how it works, thanks to chatgpt and other free resources out there.

What specific areas of your project would you like help with?

Need more help with the overflowing issues

Community feedback

  • Adriano•38,270
    @AdrianoEscarabotePosted about 1 month ago

    Hey arnoldpam11, how’s it going? I was really impressed with your project’s result, though I have some advice that could be helpful:

    Use the THE PICTURE TAG that is a shortcut to deal with the multiple images in this challenge. So you can use the <picture> tag instead of importing this as an <img> or using a div with background-image. Use it to place the images and make the change between mobile and desktop, instead of using a div or img and set the change in the css with display: none with the tag picture is more practical and easy. Note that for SEO / search engine reasons isn’t a better practice import this product image with CSS since this will make it harder to the image. Manage both images inside the <picture> tag and use the html to code to set when the images should change setting the device max-width depending of the device desktop + mobile.

    Check the link for the official documentation for <picture> in W3 SCHOOLS: https://www.w3schools.com/tags/tag_picture.asp

    See the example below:

    <picture>
  <source media="(max-width:650px)" srcset="./images/image-product-mobile.jpg">
  <img src="./images/image-product-desktop.jpg" alt="Gabrielle Parfum" style="width:auto;">
</picture>

    Everything else looks great.

    Hope this helps! 👍

    1
    arnoldpam11•120
    @arnoldpam11Posted about 1 month ago

    Thank you so much for your advice! @AdrianoEscarabote

    1
  • Sune Yde•140
    @SuneYdePosted about 1 month ago

    Its really good - just like the design!

    1
    arnoldpam11•120
    @arnoldpam11Posted about 1 month ago

    Thank you! @SuneYde

    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