Skip to content
Submitted 21 days ago

Product preview card component

RichardTagoe•70
@RichardTagoe
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?

I an really proud of how I went about the work even though it took a longer time. I learnt some new stuff and found the use of other function. The most part i am proud of is making it responsive.

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

The challenge I encountered was how to make the image and text display side by side. I watch some Youtube videos and saw how to use the flexbox and grid to conquer this challenge

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

Any suggestions and criticism are welcome since I learn from them

Community feedback

  • Adriano•38,040
    @AdrianoEscarabotePosted 20 days ago

    Hi RichardTagoe, hope you're doing well! I loved how your project turned out, but I’ve got a few suggestions that could be useful:

    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>

    The rest is fantastic.

    Hopefully, you'll find it helpful. 👍

    Marked as helpful
    0
  • RichardTagoe•70
    @RichardTagoePosted 17 days ago

    Thank you very much, Adriano Escarabote. It was very helpful.

    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