Landing page with Flex-box and Grid

Community feedback

• 

  Roy•600

  @arkaroy135Posted almost 2 years ago

  Hello Mario, Hope you are doing well. We did a good job here. However I have some suggestions for you. You did this.

  <div class="flex-box">
          <img
            src="/images/image-product-desktop.jpg"
            alt="product-image"
            class="image"
          />
  
          <img
            src="/images/image-product-mobile.jpg"
            alt="mobile=image"
            class="image-2"
          />
  

  I would like to suggest use

  <picture>
     <source media="(min-width: 1440px)" srcset="/images/image-product-desktop.jpg" aria-hidden = "true">
     <img src="/images/image-product-mobile.jpg" alt=" " aria-hidden="true">
  </picture>
  

  This will make the desktop-image appear whenever screen size is 1440px or greater, else it will show the mobile-image.

  And also use this

  <del class="price-2">$169.99</del>
  

  del tag will make the cross through exactly in the middle of the price. Feel free to ask questions if you like. Happy coding.

  1

  ---

  

  Mario Petkov•80

  @mariby21Posted almost 2 years ago

  @arkaroy135 thank your very much i try with picture element but i left it for next project i will try it :)

  1