Our reporter did not find any issues in this project! 🎉
Design comparison
Reports
Accessibility report (0)
Mustapha’s questions for the community
Hello, Frontend Mentor people! Hope you're doing great! This is my solution for the Sunnyside agency landing page.
Happy to hear any feedback and advice.
Community feedback
Ken• 975
@kenreibman
Posted
Great submission!
Your desktop layout looks great, and you did a great job making it responsive as well.
You should add a text-transform: uppercase to your header h1 to match the original design.
For your mobile layout, I initially though about limiting the width like you did with a max-width but I came to a conclusion that it looks a lot better when the content fills the entire mobile viewport width. Although it is your design choice, I believe mobile users would have an easier time reading content if it took their entire width.
Marked as helpful
1
Mustapha• 540
@muben88
Posted
@kenreibman thank you for the feedback! much appreciated! Yeah, I had the same idea about mobile design, and I totally agree with you that as a user, the content should fill the whole screen. I will try to make it. Happy coding!
0
Please log in to post a comment
Log in with GitHub
Join our Slack community
Join over 180,000 people taking the challenges, talking about their code, helping each other, and chatting about all things front-end!