Landing Page build with CSS Grid & Flexbox

Community feedback

• 

  バレンタイン 😈•64,170

  @VCaramesPosted over 2 years ago

  Good Job!

  At quick glance, I noticed that you have a major gap from your hero section and the next section.

  To fix this, remove height: 100vh from your header and use either margin or padding to add the spacing you desire.

  1

  ---

  

  arey•350

  @arey-devPosted over 2 years ago

  @vcarames I want the contents of my header to be exactly at the center viewport that's why I decided to put height: 100vh to center it easier. I tried to fix that gap without removing the height: 100vh but I wasn't successful.

  Thank you for the feedback, I'm going to wait for more from others and see their thoughts regarding that gap.

  0