Design comparison
Solution retrospective
This was very challenging but I believe my end result is close to the solution. I would love to know all changes I could've made or added. The media queries were probably the hardest. On mobile devices, I believe my media query looks OK but for ipads, tablets, etc. the image looks very big. I wasn't sure how to adjust this part without it impacting the entire media query.
Community feedback
- @benjoquilarioPosted about 3 years ago
Hey @dominiquemc Great job on finishing this one, It looks good and response very well.
Some suggestion:
- If you want a main for the main content on your page you should use the
<main> tag
instead. -I've also been at this place where you want to use the <img> tags instead of using a background image.(feels like it would be more flexible) But I realized, the custom changes you can make with background images are way more than <img> tags...so I would suggest you to use them.
Good luck and keep coding
Marked as helpful0@dominiquemcPosted about 3 years ago@benjoquilario thank you for reviewing! I'll definitely keep those tips in mind for my next challenge!
0 - If you want a main for the main content on your page you should use the
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