my-solution

Community feedback

• 

  hitmorecode•6,680

  @hitmorecodePosted about 1 year ago

  Nice looks good, just a few tips

  • Make it a habit of adding min-height: 100vh; on the body. This is important for responsiveness
  • If you remove margin on the .container and add flexbox on the body, you can place the card in the middle of the page.
  • If you prefer grid on the body add this (see below)

  body{
      font-size: var(--font-body);
      font-family: var(--font-family);
      background-color: var(--color-light-gray);
      min-height: 100vh;
      display: grid;
      place-items: center;
  }
  

  • On the Why us section you used a <p> tag and adjusted the width of the text content. There are better ways to do this. You could have use a <p> tag for each description or you could have used a <ul>

  <ul>
     <li>text</li>
     <li>text</li>
     <li>text</li>
  </ul>
  

  Marked as helpful

  1