Submitted over 2 years ago
Responsive landing page using flexbox
@AbdelrahmanTolba
Design comparison
SolutionDesign
Solution retrospective
-it not difficult
- responsive image -I need to know which part i want to improve
Community feedback
- @kkalvaaPosted over 2 years ago
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 helpful0@AbdelrahmanTolbaPosted over 2 years ago@kkalvaa thank you so much i will do my best <3
0
Please log in to post a comment
Log in with GitHubJoin 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