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 using css

@nyeno

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

Solution retrospective


I'd like to get constructive criticism on this. Thank you

Community feedback

Dharmik_487 1,740

@Dharmik48

Posted

Hey👋,

Good job on completing the challenge! But.. I found a few issues:

  • Even on big screen, the navigation bar is like it should be on mobile(the hamburger menu). Also the menu isn't working on click!
  • You have not add any hover states to any links and buttons, so add it with some transition.
  • Also change the value of media query as around 900px the content looks squeezed and even a horizontal scrollbar comes up because of the image gallery section.

Marked as helpful

0

@nyeno

Posted

@Dharmik48 Hi, thank you so much for the feedback. I've implemented the stated suggestions and will be grateful uf you could give further feedback. Thank you so much

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