Overall it looks great, lots of good stuff going on. One thing.. I used to do the changing of images just like you did in the css but then someone on frontend mentor told me about using the <picture> tag in the html instead. So for my solution I did this: <picture> <source media="(min-width: 48em)" srcset="assets/images/illustration-sign-up-desktop.svg"> <img src="assets/images/illustration-sign-up-mobile.svg" alt="main newsletter image" class=""> </picture> Your way still works, just thought I'd throw that out there. Great job!
Marked as helpful
@Juan122113
Posted
@MatthewPCope thanks for the kind words and for the advice. Do you know what of the ways is the most efficient?
@Juan122113 For most scenarios where an image is content-based and needs to adapt to different devices, the <picture> tag is more efficient. For background or decorative images that are part of the design, using a <div> with a background image in CSS might be simpler and more appropriate. Hope that helps!
Marked as helpful
