Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Sunnyside agency landing page

Mustapha 540

@muben88

Desktop design screenshot for the Sunnyside agency landing page coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
2junior
View challenge

Design comparison

SolutionDesign

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 935

@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
Discord logo

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