Landing page using html and css

Community feedback

• 

  Ahmed Bayoumi•6,740

  @Bayoumi-devPosted over 2 years ago

  Hey Olive, 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="main-page">
        //...
     </div>
  </main>
  

  • All page content should be contained by landmarks, Contain the attribution with <footer>.

  <footer>
     <div class="attribution">
        //...
     </div>
  </footer>
  

  • To handle the background image:

  body {
    font-family: "Red Hat Display", sans-serif;
    background-image: url("images/pattern-background-desktop.svg");
    font-size: 16px;
  
     background-color: hsl(225, 100%, 98%);    /* <---- Add */
     background-repeat: no-repeat;    /* <---- Add */
     background-size: contain;    /* <---- Add */
  }
  .main-page {
    height: 570px;
    width: 450px;
    border: 1px;
    border-radius: 15px;
    background-color: white;    /* <---- Changed */
    margin-top: 50px; 
    box-sizing: border-box;
    max-width: 100%; 
  }
  

  Hope this is helpful to you... Keep coding👍

  Marked as helpful

  0

  ---

  

  Olive•160

  @OlivesznPosted over 2 years ago

  @Bayoumi-dev Thank you for the correction

  0