Four Card Challenge using flexbox

Community feedback

• 

  Ahmed Bayoumi•6,740

  @Bayoumi-devPosted over 2 years ago

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

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

  <main>
     <div class="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... <h1> should represent the main heading for the whole page, and for the best practice use only one <h1> per page.

  • I suggest you use REM for font size, It is a must for accessibility because px in some browsers doesn't resize when the browser settings are changed.

  See this article ---> CSS REM – What is REM in CSS?

  Hope this help!... Keep coding👍

  Marked as helpful

  1