Kainã Mazim• 115
@KaMazim
Posted
Hi, some of your footer images are lacking alt attribute and you shouldn't use the H1 tag more than once. But apart from that, it's really good. Well done.
Marked as helpful
0
Design comparison
Solution retrospective
Feedbacks are welcome :D
Tks in advance
Community feedback
Abhijit Barman• 550
@abhijitbcob
Posted
Hi Hemanuela, In desktop view everything looks nice! But in mobile view I've noticed that you've set a width to body, and this is creating problem with responsiveness and under 375px width layout is not appearing properly.
my tips:
- set responsive width (max-width property) to elements
- don't always use fixed padding(huge padding) instead set max-width to content and center inside container
Marked as helpful
0
Hdev• 215
@hemanuela
Posted
@abhijitbcob Tks! Your feedback helped me a lot!
0
Please log in to post a comment
Log in with GitHub
Join 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