P
Matt Studdert• 13,611
@mattstuddert
Posted
Awesome work on this challenge Esu and congrats on submitting your first solution! 🥳
Here are a couple of tips after taking a look at your code:
- On mobile, the content looks very squashed. This is because of the
max-width: 50%;declaration on the.containerclass. You could change this for mobile to something likemax-width: 95%;to give more space to the content. - Have you ever tried using
min-widthmedia queries instead ofmax-width? I'd recommend giving it a go on a future project if you haven't. You typically write less code this way and also it has the benefit of loading in fewer styles for mobile users.
You've done a really good job with this. Keep up the great work! 👍
0
Esu Gabriel• 260
@orion-esu
Posted
I really appreciate Matt, i think i have worked on the observations you made. Take a look at it and see if i did the right thing.
0
P
Matt Studdert• 13,611
@mattstuddert
Posted
@EsuGabriel no problem! Nice work making the adjustments. I'd recommend increasing the breakpoint from max-width: 300px though. You can use the device emulator in Chrome Dev Tools to check out how it would look on mobile phones as a reference.
0
