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

@Abu-Sman


Design comparison


SolutionDesign

Solution retrospective


I need some feedbacks fam!

Community feedback

P

@vickievien

Posted

Design looks near flawless! I would include more semantic HTML by breaking out the code you have, instead of having everything within <header></header>.

At a high level, something like this:

<body>
  <header>
      [HUDDLE LOGO CODE GOES HERE]
  </header>
  <main>
      [ILLUSTRATION MOCKUP & TEXT BOX GOES HERE]
  </main>
  <footer>
      <ul>
          <li>[SOCIAL MEDIA ICON]</li>
          <li>[SOCIAL MEDIA ICON]</li>
          <li>[SOCIAL MEDIA ICON]</li>
      </ul>
  </footer>
</body>

This way, it'll help define the different sections better and help with SEO. Hope this helps.

Happy coding!

Marked as helpful

0

@Abu-Sman

Posted

I feel you @vickievien. I assumed this was a header of a large website, which is why i never considered splitting it up. But i will definitely keep this in mind when i build something similar in the future. 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