Order Summary

Community feedback

• 

  Ahmed Bayoumi•6,740

  @Bayoumi-devPosted over 2 years ago

  Hey Jason, It looks good!... You have accessibility issues that need to fix.

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

  <main>
     <div class="main-container">
        //...
     </div >
  </main>
  

  • Page should contain a level-one heading, Change h2 to h1 You should always have one h1 per page of the document... in this challenge, you will use h1 just to avoid the accessibility issue that appears in the challenge report... but don't use h1 on small components <h1> should represent the main heading for the whole page, and for the best practice use only one <h1> per page.

  • Heading levels should only increase by one, Use h2 instead of h4... Ordered headings make it easier to navigate and understand when using assistive technologies.

  • I suggest you add cursor: pointer; to buttons, The cursor indicates to users there is an action that will be executed when clicking on it.

  I hope this is useful to you... Keep coding👍

  0

  ---

  

  Jason•140

  @jsngmbPosted over 2 years ago

  Hi @Bayoumi-dev I really appreciate your willingness to help me. Thank you so much.

  0