Responsive landing page using flex-box, css grid and media query

Community feedback

• 

  Ahmed Bayoumi•6,700

  @Bayoumi-dev

  Posted almost 3 years ago

  Hey Ramsès, Congratulations on completing this challenge... You have accessibility issues that need to fix.

  • Document should have one main landmark, Contain the main content with <main>.

  <main>
     <section id="hero">
        //...
     </section>
  
     <section id="services">
        //...
     </section>
  
     <section id="articles">
        //...
     </section>
  </main>
  

  • Heading levels should only increase by one, Use h4 instead of h5... Always Make sure you are using the headings in order, Ordered headings make it easier to navigate and understand when using assistive technologies.

  Hope this help!... Keep coding👍

  00

  Marked as helpful
• 

  Ramsès Jr•30

  @codebypharaon

  Posted almost 3 years ago

  Hello Ahmed Thanks for your feedback, I saw these issues...I'm going to fix that, then update my solution.

  There is a lot of things to learn here so I'm always going to keep CODING. Thanks...

  00