Responsive product preview card using CSS flexbox

Community feedback

• 

  Shaswat Singh•190

  @shaswatsingh19Posted over 2 years ago

  Nice work, just that you should reduce your whole container width so that it will be perfect as the design. You also did mobile first design which is mostly preferred and that's great. You can watch Kevin Powell's video on width and max-width it is really great resource. I saw your code and you used both images inside the img div and put one to none and other one to display block. I would suggest to use both images as background-image and get them as url() it reduces the code and you didn't have to display block or none the the csss

  Marked as helpful

  0

  ---

  

  Mahad Aamir•100

  @mahadaamir35Posted over 2 years ago

  @shaswatsingh19 Thanks for the feedback bro. Kevin Powell's videos are truly resourceful and effective. I will surely try to use the background-image property in my future projects.

  0