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

Huddle landing page with single introductory section

@Banjer71


Design comparison


SolutionDesign

Community feedback

P
Matt Studdert 13,611

@mattstuddert

Posted

Looks great Davide!

I noticed a few things while having a look through your code:

  • In your CSS you don't need to use the * selector for properties such as font-size, color and font-family, as these properties can make use of CSS inheritance. Here's a great article about it if you want to read more.

  • The background-image on the body could do with having background-size: cover added to it to make sure it stretches to fit the screen.

  • Nice work using a min-width media query. This helps keep your loaded styles as lean as possible for mobile users.

  • Quick heads up that you've also got a random closing footer tag in your HTML without an opening tag.

  • Great use of HTML5 semantic elements as well 👍

Hope that helps. Keep up the great work! 🙂

0

@Banjer71

Posted

Thank you very much Matt. I will fix the things you mentioned. ;)

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