Responsive landing page using css flexbox

Community feedback

• 

  Ecem Gokdogan•9,380

  @ecemgoPosted over 1 year ago

  Some recommendations regarding your code that could be of interest to you.

  HTML

  In order to fix the accessibility issues:

  • You need to replace <div class="main"> with the <main class="main"> tag. You'd better use Semantic HTML, and you can also reach more information about it from Using Semantic HTML Tags Correctly.

  After committing the changes on GitHub and you need to deploy it as a live site. Finally, you should click generate a new report on this solution page to clear the warnings.

  CSS

  • For the color of the screen, you can use the recommended color in the body

  body {
    /* background-color: hsl(220, 15%, 55%);   */
    background-color: hsl(212, 45%, 89%);
  }
  

  Hope I am helpful. :)

  Marked as helpful

  0