Design comparison
Solution retrospective
I managed to make it responsive.
Community feedback
- Account deleted
👋 Hey, Jonas! I have a few suggestions for your project:
Use
<picture>
for Multiple Images<picture class="image"> <source srcset="images/desktop.jpg" media="(min-width: 768px)"> <img src="images/mobile.jpg" alt="Product image"> </picture>
You no longer need to use
display: none
; the browser will take care of which image to display.Use
<footer>
Instead of<header>
for .attributionUse
<section>
for Grouping Related Content For example, the.information
div could be wrapped in a<section>
element.<section class="information"> ... </section>
Use Classes (.class) Instead of IDs (#id) for Styling Use IDs when working with JavaScript or accessibility.
More info:
I hope this helps you! 😊
Marked as helpful0@jonaschaconPosted 3 months agoThanks for the suggestions, they will help me a lot! 😊 @weldu0
1Account deletedYou're very welcome! I'm glad to hear that!😊 @jonaschacon
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