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 a single introductory section using flex

@NJR911


Design comparison


SolutionDesign

Solution retrospective


I think that's my worst challenge, if you have any advices please i'm listening

Community feedback

@BilalSalmi

Posted

Hi Mohamed, good job! your design is so good but still need to be optimized.

Accessibility recommadations

  • Your web page should contain just one <main> element

  • consider using Html semantic elements like <section> <article> <header> <nav> <footer> in your page.

Read more about them here.

  • consider to use html h1 - h6 for your page headings

You can read more about them here.

  1. style ideas
  • your paragraphe's vertical spaces between lines need to increased, for that you can use css line-height .

line-height: 1.8

Keep it up!

Marked as helpful

0

@NJR911

Posted

@BilalSalmi thanks for your feedback, i'll try to do that next time

thank you!

0
Thais 245

@thaiscode

Posted

Hi Mohamed, you did a great job!

Excessive use of <div> is not a good practice, take a look at semantic html.

Reference link: https://developer.mozilla.org/en-US/docs/Learn/Accessibility/HTML

Marked as helpful

0

@NJR911

Posted

@thaiscode I really appreciate that

1

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