Skip to content
Submitted 4 months ago

Responsive Product Review Card

mamman-naf•130
@mamman-naf
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?

This project was really tricky but I'm i did justice to it. well next time when dealing with anything that has to do with responsiveness i will first of all design the mobile version first before doing the desktop version.

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

using media queries was a bit of challenging, but then after i understood the rules on it, it became as if it was an if and else statement

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

would appreciate it very much to know more about media queries tips and tricks

Community feedback

  • Adriano•42,800
    @AdrianoEscarabotePosted 4 months ago

    Hi mamman-naf, how’s everything? I think your project turned out great! However, I have some feedback that I think might 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 amazing.

    I hope this is helpful. 👍

    0
    mamman-naf•130
    @mamman-nafPosted 4 months ago

    @AdrianoEscarabote I'm good. Thanks for the feedback. will make sure to work on that. but i tried using the </picture> tag, it just didn't work out good so i took the easier route. But I shall give the example you share with me a try. this really was helpful. THANK YOU VERY MUCH

    1

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