Design comparison
SolutionDesign
Solution retrospective
Tried developing mobile first on this. Let me know how I did.
Community feedback
- @mattstuddertPosted over 4 years ago
Nice work, Herman! Your solution looks really good and it's great to see you using Flexbox. Here are a couple of pointers after taking a look at your code:
- You've got
width: 750px;
on the.container
class. This is fine, but it means that you get a horizontal scrollbar on screen sizes smaller than750px
. You could add amax-width: 95%;
to that class to get it to scale down when the screen size gets too small. You'd also then need to addmargin: 0 auto
to center it. - You've also got
width: 375px;
on the inner elements. Instead of using pixels for widths like this, I'd recommend using percentages wherever possible. This will help with building responsive websites as the elements will scale up and down with the screen.
I hope that helps. Keep up the great work!
0 - You've got
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