Responsive landing page using CSS Flexbox

Community feedback

• 

  Eileen dangelo•1,600

  @EileenpkPosted almost 2 years ago

  Hi Alex, congrats on your first Frontend Mentor project!!!! 🎉🎉🎉 Your project looks great and very clean. I did notice something small when I was looking at your HTML and I thought it might be helpful to point it out. Your <main class='container'></main> is this outer wrapper correct? You then have a <div class='card'></div> that is inside that main tag. When you closed off <main>, the closing tag of the </div> should be inside the </main> tag for proper nesting and readability. This will also help prevent bugs. It should look like this-

  <main class='container'>
      <div class='card'>
  </div>
  </main>
  

  Hope this helps! If you found this helpful please mark it as such :)

  • Let's connect on LinkedIn! - @Eileenpk

  Marked as helpful

  0

  ---

  

  Alex•20

  @GoodAlex223Posted almost 2 years ago

  Hello Eileen, thanks for your comment! I really closed the tag in the wrong place, overlooked) Thank you again, next time I will be more careful!

  0