Design comparison
Solution retrospective
I did it quite quickly and with no many problems.
What challenges did you encounter, and how did you overcome them?Aligning the content horizontally in desktop version. I used flexbox display row to solve it.
What specific areas of your project would you like help with?How to improve the look and efficiency of the code.
Community feedback
- @Alex-Archer-IPosted 3 months ago
Hey!
Great work, but I really think that you should use
img
tag here instead of background image. You see, backgrounds good for decorative pictures which doesn't bear any context meanings. And theimg
have analt
attribute with description for assistive technologies.To switch images in the different versions you can use
picture
tag.<picture> <source srcset="mobile.jpg" media="(max-width: 900px)"> <img src="desktop.jpg" alt=""> </picture>
It contains one (or more)
source
tags with different media conditions. If no one of the medias will match, theimg
would be rendered.Oh, and also all your media queries have
min-width: 1024px
. They would be overwrite each other that way =)Marked as helpful1
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