Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Responsive landing page using flexbox

@AbdelrahmanTolba

Desktop design screenshot for the Product preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


-it not difficult

  • responsive image -I need to know which part i want to improve

Community feedback

@kkalvaa

Posted

For responsive images you should look into srcset instead of having two <img> and hiding one with css. When you hide images in css the browser will still load the image, but only show you one. In other words you end up using more bandwidth than if you just sent over the big one and scaled it down.

Instead of "card__left" you should call it "card__image" or similar. Since the image isn't on the left in mobile view "card__left" doesn't make sense anymore, but "card__image" always will.

Marked as helpful

0

@AbdelrahmanTolba

Posted

@kkalvaa thank you so much i will do my best <3

0

Please log in to post a comment

Log in with GitHub
Discord logo

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