responsive using css flexbox, grid and @media

Community feedback

• 

  Ahmed Bayoumi•6,740

  @Bayoumi-devPosted over 2 years ago

  **Hey!**It looks good!...

  My suggestions:

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

  <main>
     <section class="socialproof">
        //...
     </section>
  </main>
  

  • Using more than one <h1> is allowed by the HTML specification, but is not considered a best practice. Using only one <h1> is beneficial for screenreader users, and <h1> should represent the main heading for the whole page

  ---> [Multiple <h1> elements on one page](https://developer.mozilla.org/en-US/docs/Web/HTML/Element/Heading_Elements#multiple_h1_elements_on_one_page

  • All page content should be contained by landmarks, Contain the attribution with <footer>. Hope this help!... Keep coding👍

  Marked as helpful

  0