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

Desktop design screenshot for the Huddle landing page

wojt wojt 60

@caren1


Design comparison


SolutionDesign

Community feedback

P
Matt Studdert 13,611

@mattstuddert

Posted

Nice work on this! A few small things I saw when having a look at your code:

  • You've currently got this code in your CSS
.main-heading {
    width: 50vh;
}

I would be careful setting an element's width using the height of the viewport. It can lead to some funny element dimensions if the user reduces the height of the browser.

  • In your HTML you currently have a lot of div elements. I would definitely say that there could be some more semantic HTML5 structural elements that could be used like header, footer, main and section in this instance.
  • Your media queries don't need to be as specific as they currently are. I'd recommend just using min-width and keeping it as simple as possible at first.

Keep up the great work! 👍

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